Skip to content

Manual backport of [NET-6741] make: Add target for updating dependencies across all modules into release/1.15.x#20687

Merged
zalimeni merged 2 commits intorelease/1.15.xfrom
backport/zalimeni/net-6741-add-make-target-dependency-update-modules-1.15
Feb 27, 2024
Merged

Manual backport of [NET-6741] make: Add target for updating dependencies across all modules into release/1.15.x#20687
zalimeni merged 2 commits intorelease/1.15.xfrom
backport/zalimeni/net-6741-add-make-target-dependency-update-modules-1.15

Conversation

@zalimeni
Copy link
Member

@zalimeni zalimeni commented Feb 21, 2024

Backport of #19785 which also includes updates to Go modules list / Make target from #17462 (landed on main during 1.16).

Asking for a quick extra review bc @rboyer 's original go-mod-tidy introduction noted "the correct order" and I wanted to make sure this wasn't sneakily risky in 1.15.

Testing & Reproduction steps

Tested locally as in original PR, same results.

PR Checklist

  • updated test coverage
  • external facing docs updated
  • appropriate backport labels added
  • not a security concern
Overview of commits

Update go-mod-tidy Make target to use flexible modules list.

This will update the list to include envoyextensions and troubleshoot.
GOLANGCI_LINT_VERSION='v1.55.2'
MOCKERY_VERSION='v2.20.0'
BUF_VERSION='v1.4.0'

Copy link
Member Author

Choose a reason for hiding this comment

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

Propagated from later branches intentionally to aid future backports.

@zalimeni zalimeni merged commit 4a88465 into release/1.15.x Feb 27, 2024
@zalimeni zalimeni deleted the backport/zalimeni/net-6741-add-make-target-dependency-update-modules-1.15 branch February 27, 2024 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr/no-backport pr/no-changelog PR does not need a corresponding .changelog entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants