Add composite GitHub Action for modctl build workflow (#507)#508
Add composite GitHub Action for modctl build workflow (#507)#508rishi-jat wants to merge 2 commits intomodelpack:mainfrom
Conversation
Signed-off-by: Rishi Jat <rishijat098@gmail.com>
There was a problem hiding this comment.
Code Review
This pull request introduces a composite GitHub Action to automate the installation of modctl and the building of model artifacts, accompanied by updated documentation in the README and getting started guide. The review feedback identifies critical security risks related to shell injection and secret exposure when using GitHub Action expressions directly within shell scripts, advising that inputs and secrets be mapped to environment variables. Furthermore, there is a recommendation to replace brittle JSON parsing logic with more robust tools like the GitHub CLI or jq.
There was a problem hiding this comment.
Pull request overview
This PR introduces a composite GitHub Action (at the repo root) to install modctl and run modctl build in workflows, with optional registry login support, and adds docs + a CI workflow to exercise the action.
Changes:
- Added a root-level composite action (
action.yml) that installsmodctl, optionally logs into a registry, and builds a model artifact. - Added a dedicated workflow to validate the action against “latest” and a pinned
modctlversion and to execute the registry-login path. - Updated README and getting-started docs with usage examples and input reference.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
action.yml |
New composite action that validates inputs, installs modctl from GitHub Releases, optionally logs in, and runs modctl build. |
docs/getting-started.md |
Added a “GitHub Action” section documenting usage, inputs, version pinning, and optional registry integration. |
README.md |
Added a minimal action usage snippet and link to the detailed docs section. |
.github/workflows/modctl-action.yml |
Added CI workflow to exercise local action usage with latest/pinned versions and the registry-login execution path. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Rishi Jat <rishijat098@gmail.com>
|
/cc @sabre1041 |
Summary
This change introduces a composite GitHub Action to streamline the use of modctl in GitHub Actions workflows.
The action installs modctl (latest or pinned version), builds a model artifact from a Modelfile, and optionally performs registry authentication for remote artifact output.
Key points:
Validation:
Documentation:
Notes:
Fixes #507