health action: let someone point at a different ref/repo#411
Conversation
PR HealthChangelog Entry ✔️
Changes to files need to be accounted for in their respective changelogs. This check can be disabled by tagging the PR with Coverage ✔️
This check for test coverage is informational (issues shown here will not fail the PR). This check can be disabled by tagging the PR with Unused Dependencies ✔️
For details on how to fix these, see dependency_validator. This check can be disabled by tagging the PR with Breaking changes ✔️
This check can be disabled by tagging the PR with API leaks ✔️The following packages contain symbols visible in the public API, but not exported by the library. Export these symbols or remove them from your publicly visible API.
This check can be disabled by tagging the PR with License Headers ✔️
All source files should start with a license header. This check can be disabled by tagging the PR with |
There was a problem hiding this comment.
Code Review
This pull request introduces a new GitHub Action for installing ecosystem packages using a composite action and a supporting Bash script. The feedback suggests enhancing the script's robustness by adding error handling with set -eo pipefail, validating the presence of WORKFLOW_REF, and supporting short repository names. Additionally, it is recommended to quote the script path in the action definition to prevent issues with spaces in directory names.
Allows more experimenting
ad13211 to
1d4da9c
Compare
Allows more experimenting