-
Notifications
You must be signed in to change notification settings - Fork 10
Support the installation and use of act to test and run GitHub workflows
#222
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support the installation and use of act to test and run GitHub workflows
#222
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR adds support for installing and using act, a tool for running GitHub Actions workflows locally. The changes enable developers to test workflows without pushing to GitHub.
- Adds an installation script for the
actbinary with architecture detection and error handling - Provides default configuration for running
acton the host machine - Updates .gitignore to exclude act-generated artifacts and secrets
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated 7 comments.
| File | Description |
|---|---|
| scripts/install-act.sh | Installation script that downloads and installs the latest act release from GitHub with architecture detection for x86_64 and arm64 systems |
| .actrc | Configuration file specifying default container images, architecture, and artifact storage paths for local workflow testing |
| .gitignore | Adds entries to ignore act-generated artifacts directory and secrets file |
|
@greenc-FNAL, why should the phlex repository have an |
As a development helper: it can be used to install |
I'm skeptical that developer helpers like So, I think the |
Updated the text formatting standard to explicitly state that the final line must be non-empty and terminated with a single linefeed, leaving no trailing blank lines. This clarification addresses potential ambiguity in the previous wording. Co-authored-by: greenc-FNAL <2372949+greenc-FNAL@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 2 out of 3 changed files in this pull request and generated 2 comments.
Per @copilot review. Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
No description provided.