chore: bump go, tools, deps and re-kres, satisfy linters#531
Merged
Conversation
There was a problem hiding this comment.
Pull Request Overview
Bump Go version, dependencies, tool versions, and update code to satisfy new linter checks.
- Added
//nolint:embeddedstructfieldcheckcomments on embeddeddag.BaseNodefields - Reordered struct fields in several modules
- Updated version constants,
go.mod, Dockerfile syntax, Makefile, and CI workflows
Reviewed Changes
Copilot reviewed 19 out of 20 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| internal/project/js/build.go | Reordered meta and dag.BaseNode fields in Build |
| internal/project/common/release.go | Moved meta below dag.BaseNode in Release |
| internal/output/ghworkflow/types.go | Reordered Name and Schedule fields in workflow types |
| internal/config/constants.go | Bumped various image and tool version constants |
| go.mod | Updated Go version and bumped dependency versions |
| Dockerfile | Upgraded frontend image syntax and bun version |
Comments suppressed due to low confidence (4)
internal/project/js/build.go:26
- The order of struct fields was changed by moving
metabelowdag.BaseNode. This alters the memory layout and may affect initialization or behavior—please confirm this reordering is intentional or revert to the original order.
meta *meta.Options
internal/project/common/release.go:26
- Reordering the
metafield belowdag.BaseNodechanges the struct layout. Verify that this change won't break serialization or upstream consumers.
meta *meta.Options
internal/output/ghworkflow/types.go:14
- Moving the
Namefield changes the order of keys in generated YAML. Ensure this reordering won't break existing GitHub workflow expectations.
Name string `yaml:"name"`
internal/output/ghworkflow/types.go:32
- [nitpick] Reordering
Schedulewithin theOnstruct can change the sequence of fields in the output YAML—please confirm it's intentional.
Schedule []Schedule `yaml:"schedule,omitempty"`
frezbo
approved these changes
Jul 11, 2025
Unix4ever
approved these changes
Jul 11, 2025
Bump everything. Signed-off-by: Utku Ozdemir <utku.ozdemir@siderolabs.com>
9b39fa4 to
c691b83
Compare
Member
Author
|
/m |
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.
Bump everything.