Skip to content

Update go and modules #1032

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Update go and modules #1032

wants to merge 4 commits into from

Conversation

mouchar
Copy link

@mouchar mouchar commented Jul 25, 2025

Checklist
  • make test-all (UNIX) passes. CI will also test this
  • unit and/or integration tests are included (if applicable)
  • documentation is changed or added (if applicable)

Description of change

  • Update go and modules to address CVEs (CVE on goss #1008)
  • One test fixture had to be adjusted to cover change in gomega module
  • Updated golangci linter (old version failed on internal error on "unicode" module)
  • Fixed minor finding from updated staticcheck

mouchar added 4 commits July 25, 2025 11:04
@mouchar
update go and modules
0affea0
@mouchar
Adjust tests to new gomega version
189d6d5 
Gomega 1.36 supports iterators. Update test fixtures.
@mouchar
upgrade golangci-lint
ae31238 
Convert config file, preserving previous settings.
Explicitly disable ST1005 on top of defaults
@mouchar
Address staticcheck findings
2269cc9
@mouchar mouchar requested a review from aelsabbahy as a code owner July 25, 2025 09:20
@kgaughan kgaughan mentioned this pull request Sep 14, 2025
Open
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@aelsabbahy aelsabbahy Awaiting requested review from aelsabbahy aelsabbahy is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@mouchar