Skip to content

Fix upgrade workflow#31

Open
silv-io wants to merge 2 commits intomainfrom
fix-go-upgrade-workflow
Open

Fix upgrade workflow#31
silv-io wants to merge 2 commits intomainfrom
fix-go-upgrade-workflow

Conversation

@silv-io
Copy link
Member

@silv-io silv-io commented Feb 13, 2026

The new upgrade workflow installed the current go version to upgrade the go version, which doesn't work in CI by the looks of it (works locally though).

Should've done a test run before merging. Fixed it now and confirmed with test runs that it will work in #32 and #33

@silv-io silv-io marked this pull request as ready for review February 13, 2026 19:28
@coderabbitai
Copy link

coderabbitai bot commented Feb 13, 2026

📝 Walkthrough

Walkthrough

Updated the weekly Go upgrade GitHub Actions workflow to use stable Go version with latest check enabled, configured PR author and committer bot details, and replaced branch deletion with explicit token-based authentication and specific reviewer assignment.

Changes

Cohort / File(s) Summary
Go Upgrade Workflow Configuration
.github/workflows/weekly-go-upgrade.yml
Modified Go setup to use stable version with check-latest enabled, expanded Create Pull Request step with bot author/committer fields, replaced delete-branch flag with PRO_ACCESS_TOKEN authentication, and added reviewers list.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • carole-lavillonniere
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Fix upgrade workflow' is directly related to the changeset, which fixes the Go upgrade workflow by correcting the version setup and CI configuration.
Description check ✅ Passed The description is related to the changeset, explaining that it fixes an issue in the upgrade workflow where the wrong Go version was being used in CI.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into main

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix-go-upgrade-workflow

No actionable comments were generated in the recent review. 🎉


Comment @coderabbitai help to get the list of available commands and usage tips.

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.

1 participant