ci: unify CI and release workflows into nanvix-ci.yml#54
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
Consolidates the previously separate ci.yml and release.yml GitHub Actions workflows into a single nanvix-ci.yml, with per-job concurrency groups and shared env vars.
Changes:
- Adds
buildjob (PRs + pushes todev) withci-*concurrency group allowing cancellation of stale runs - Keeps
releasejob (repository_dispatch) withrelease-gccconcurrency group, no cancellation - Centralizes
GCC_VERSION,INSTALL_LOCATION, andGH_TOKENat workflow level
Show a summary per file
| File | Description |
|---|---|
| .github/workflows/nanvix-ci.yml | New unified workflow combining build and release jobs with per-job concurrency and gated if conditions. |
| .github/workflows/ci.yml | Deleted; functionality merged into nanvix-ci.yml. |
Note: release.yml is referenced in the PR description as being merged in, but it does not appear in the diff. The unified file appears to be derived from the old release.yml (note line numbers 110+ for the release job match its prior content), so the deletion of release.yml is presumably also part of this PR but not shown here.
Copilot's findings
- Files reviewed: 2/2 changed files
- Comments generated: 0
Merge the separate ci.yml and release.yml workflow files into a single nanvix-ci.yml. The build job handles PRs and pushes to dev, while the release job handles repository_dispatch events. Shared environment variables are defined once at the workflow level.
ppenna
force-pushed
the
enhancement/unify-ci-release-workflow
branch
from
3fbcba2 to
797d93e
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Merge the separate
ci.ymlandrelease.ymlworkflow files into a singlenanvix-ci.yml.Changes:
buildjob handles PRs and pushes todev(previouslyci.yml)releasejob handlesrepository_dispatchevents (previouslyrelease.yml)GCC_VERSION,INSTALL_LOCATION,GH_TOKEN) are defined once at the workflow levelci-*for builds (allows cancellation of stale PR runs) andrelease-gccfor releases (no cancellation)