Skip to content

Conversation

@Brian-McM
Copy link
Contributor

Description

This commit adds a new utility, gomodder, to help verify and modify our various go.mods to initiall ensure:

  • Each module follows specific import restrictions
  • Verify go module versions are in sync

As follow up, the tool can be modified to identify version mismatches and automatically update go.mods.

Related issues/PRs

Todos

  • Tests
  • Documentation
  • Release note

Release Note

TBD

Reminder for the reviewer

Make sure that this PR has the correct labels and milestone set.

Every PR needs one docs-* label.

  • docs-pr-required: This change requires a change to the documentation that has not been completed yet.
  • docs-completed: This change has all necessary documentation completed.
  • docs-not-required: This change has no user-facing impact and requires no docs.

Every PR needs one release-note-* label.

  • release-note-required: This PR has user-facing changes. Most PRs should have this label.
  • release-note-not-required: This PR has no user-facing changes.

Other optional labels:

  • cherry-pick-candidate: This PR should be cherry-picked to an earlier release. For bug fixes only.
  • needs-operator-pr: This PR is related to install and requires a corresponding change to the operator.

@Brian-McM Brian-McM requested a review from a team as a code owner May 23, 2025 21:41
@marvin-tigera marvin-tigera added this to the Calico v3.31.0 milestone May 23, 2025
@marvin-tigera marvin-tigera added release-note-required Change has user-facing impact (no matter how small) docs-pr-required Change is not yet documented labels May 23, 2025
@Brian-McM Brian-McM added docs-not-required Docs not required for this change release-note-not-required Change has no user-facing impact and removed release-note-required Change has user-facing impact (no matter how small) labels May 23, 2025
@marvin-tigera marvin-tigera removed the docs-pr-required Change is not yet documented label May 23, 2025
@Brian-McM Brian-McM force-pushed the bm-add-utility-to-verify-go-modules branch 3 times, most recently from 8eb4eb0 to 09027cb Compare May 29, 2025 17:48
This commit adds a new utility, gomodder, to help verify and modify our various go.mods to initiall ensure:
- Each module follows specific import restrictions
- Verify go module versions are in sync

As follow up, the tool can be modified to identify version mismatches and automatically update go.mods.
@Brian-McM Brian-McM force-pushed the bm-add-utility-to-verify-go-modules branch from 09027cb to bad5301 Compare May 30, 2025 16:51
Copy link
Member

@caseydavenport caseydavenport left a comment

Choose a reason for hiding this comment

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

seems neat to me, happy to give it a go (mod)

@Brian-McM Brian-McM merged commit cd4e902 into projectcalico:master May 30, 2025
4 checks passed
@Brian-McM Brian-McM deleted the bm-add-utility-to-verify-go-modules branch May 30, 2025 20:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs-not-required Docs not required for this change release-note-not-required Change has no user-facing impact

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants