Skip to content

Conversation

synarete
Copy link
Collaborator

Update project to newer golang version an tools. Use golang 1.24 for container image.

Upgrade project to support the latest two version of golang.

Signed-off-by: Shachar Sharon <[email protected]>
Copy link
Collaborator

@phlogistonjohn phlogistonjohn left a comment

Choose a reason for hiding this comment

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

looks ok to me.

Copy link
Contributor

@anoopcs9 anoopcs9 left a comment

Choose a reason for hiding this comment

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

See below for a nit.

Otherwise lgtm, thanks.


_install_yq() {
_install_tool github.com/mikefarah/yq/v4@v4.23.1
_install_tool github.com/mikefarah/yq/v4@v4.45.3
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: upstream says v4.45.4 as the latest version.

synarete added 5 commits May 19, 2025 18:21
Using latest yq release.

Signed-off-by: Shachar Sharon <[email protected]>
Using latest 'revive' release.

Signed-off-by: Shachar Sharon <[email protected]>
Upgrade to latest golangci-lint version. Used migration-guide[1] to
adjust golangci configuration file to new version.

[1] https://golangci-lint.run/product/migration-guide/

Signed-off-by: Shachar Sharon <[email protected]>
Align github workflow with recommended[1] setup-go and go-version
settings.

[1] https://github.com/actions/ \
      setup-go?tab=readme-ov-file#using-stableoldstable-aliases

Signed-off-by: Shachar Sharon <[email protected]>
Update to latest (from Feb 2025) golang version.

Signed-off-by: Shachar Sharon <[email protected]>
@synarete synarete force-pushed the ss-project-update branch from 6cb04e9 to 511f6f9 Compare May 19, 2025 15:22
@synarete synarete merged commit 4577a70 into samba-in-kubernetes:main May 19, 2025
7 checks passed
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.

4 participants