Skip to content

Remove unused replace directives#2265

Merged
matttrach merged 2 commits into
rancher:mainfrom
JonCrowther:remove-unused-imports
May 27, 2026
Merged

Remove unused replace directives#2265
matttrach merged 2 commits into
rancher:mainfrom
JonCrowther:remove-unused-imports

Conversation

@JonCrowther
Copy link
Copy Markdown
Contributor

Addresses #2255

Description

There are a few replace directives that aren't actually used within the codebase. go mod tidy doesn't look to remove replace directives so this needs to be done manually.

Testing

Not a breaking change.

Copilot AI review requested due to automatic review settings May 27, 2026 13:04
@JonCrowther JonCrowther self-assigned this May 27, 2026
@JonCrowther JonCrowther requested a review from matttrach May 27, 2026 13:05
@JonCrowther
Copy link
Copy Markdown
Contributor Author

With the additional gorilla/mux change, this also addresses #2040

@matttrach matttrach added release/v14 Targets release v14.x correlating to Rancher's v2.14.x release/v15 Targets release v15.x correlating to Rancher's v2.15.x labels May 27, 2026
@matttrach matttrach merged commit 8a8e27a into rancher:main May 27, 2026
8 of 9 checks passed
@matttrach matttrach removed the release/v14 Targets release v14.x correlating to Rancher's v2.14.x label May 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release/v15 Targets release v15.x correlating to Rancher's v2.15.x

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants