Fix CI: update Go version and GitHub Actions to current versions#11
Fix CI: update Go version and GitHub Actions to current versions#11renecannao merged 1 commit intomasterfrom
Conversation
|
Note Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported. |
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull request overview
Updates GitHub Actions workflows to align CI with the Go version declared in go.mod and modern supported major versions of common GitHub Actions.
Changes:
- Bump workflow Go toolchain from 1.16 to
1.25.7viaactions/setup-go@v5. - Update
actions/checkoutto@v4across workflows andactions/upload-artifactto@v4inmain.yml. - Switch
orchestrator-ci-envclone URL fromopenarktoperconainsystem.yml.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| .github/workflows/main.yml | Updates checkout/setup-go/upload-artifact versions and Go toolchain version used in CI. |
| .github/workflows/system.yml | Updates checkout/setup-go versions, Go toolchain version, and orchestrator-ci-env repo URL. |
| .github/workflows/upgrade.yml | Updates checkout/setup-go versions and Go toolchain version for upgrade workflow. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| steps: | ||
| - uses: actions/checkout@v2 | ||
| - uses: actions/checkout@v4 | ||
|
|
||
| - name: Set up Go 1.16 | ||
| uses: actions/setup-go@v1 | ||
| - name: Set up Go | ||
| uses: actions/setup-go@v5 | ||
| with: | ||
| go-version: 1.16 | ||
| go-version: '1.25.7' |
There was a problem hiding this comment.
The steps: list items are indented at the same level as steps: (e.g., - uses: actions/checkout@v4). YAML requires the sequence items to be indented under steps:; as written this workflow is likely invalid and will fail to parse in GitHub Actions. Please indent all step entries (and their nested keys) consistently under steps:.
| steps: | ||
| - name: Set up Go 1.16 | ||
| uses: actions/setup-go@v1 | ||
| - name: Set up Go | ||
| uses: actions/setup-go@v5 | ||
| with: | ||
| go-version: 1.16 | ||
| go-version: '1.25.7' | ||
|
|
||
| - name: check out | ||
| uses: actions/checkout@v2 | ||
| uses: actions/checkout@v4 | ||
|
|
There was a problem hiding this comment.
The steps: entries are not indented under steps: (the - name: line starts at the same indentation level as steps:). This makes the YAML structure invalid and will likely prevent the workflow from running. Please indent the entire steps sequence (and nested keys) under steps:.
| steps: | ||
| - name: Set up Go 1.16 | ||
| uses: actions/setup-go@v1 | ||
| - name: Set up Go | ||
| uses: actions/setup-go@v5 | ||
| with: | ||
| go-version: 1.16 | ||
| go-version: '1.25.7' | ||
|
|
There was a problem hiding this comment.
The steps: list items are aligned with steps: instead of being indented beneath it. GitHub Actions workflows are strict YAML; this indentation is likely to make the workflow fail to load. Please indent all - name: / - uses: items (and their nested keys) under steps:.
Summary
actions/checkoutfrom v2 to v4actions/setup-gofrom v1 to v5actions/upload-artifactfrom v1 to v4 (in main.yml)orchestrator-ci-envclone URL from openark to percona (in system.yml)Test plan
@v1,@v2, orgo-version: 1.16in workflow filesCloses #5