Skip to content

Change checkout ref from 'stable' to 'dev'#27

Merged
hverhelst merged 1 commit into
mainfrom
hotfix/checkout-gismo-in-dev
Mar 3, 2026
Merged

Change checkout ref from 'stable' to 'dev'#27
hverhelst merged 1 commit into
mainfrom
hotfix/checkout-gismo-in-dev

Conversation

@hverhelst
Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings March 3, 2026 14:43
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the CI workflow’s external dependency checkout to follow the dev branch of gismo/gismo, changing which upstream code is used during CI runs.

Changes:

  • Change actions/checkout of gismo/gismo from ref: stable to ref: dev.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/ci.yml
with:
repository: gismo/gismo
ref: stable
ref: dev
Copy link

Copilot AI Mar 3, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Switching the dependency checkout from ref: stable to ref: dev makes CI depend on a potentially more volatile upstream branch. If the goal isn’t explicitly to test against gismo/gismo’s latest dev, consider pinning to a tag or commit SHA (or documenting why dev is required) to keep CI runs reproducible.

Suggested change
ref: dev
ref: stable

Copilot uses AI. Check for mistakes.
Comment thread .github/workflows/ci.yml
@@ -26,7 +26,7 @@ jobs:
- uses: actions/checkout@master
Copy link

Copilot AI Mar 3, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

actions/checkout@master tracks the action’s default branch, which can change unexpectedly and is discouraged for security/reproducibility. Pin this to a stable release (e.g., actions/checkout@v4) or a specific commit SHA instead of @master.

Copilot uses AI. Check for mistakes.
@hverhelst hverhelst merged commit 59c7403 into main Mar 3, 2026
7 checks passed
@hverhelst hverhelst deleted the hotfix/checkout-gismo-in-dev branch March 3, 2026 15:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants