Skip to content
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.

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

x/tools/gopls: handle conflicting replaces in the workspace module #44698

Closed
stamblerre opened this issue Feb 28, 2021 · 1 comment
Closed

x/tools/gopls: handle conflicting replaces in the workspace module #44698

stamblerre opened this issue Feb 28, 2021 · 1 comment
Labels
gopls/workspace gopls Tools
Projects
gopls on-deck
Milestone
gopls/later

Comments

@stamblerre
Copy link
Contributor

stamblerre commented Feb 28, 2021 •
edited

When the workspace module replaces 2+ different versions of the same dependency, we should prefer the highest version. We should also prefer file replaces to version replaces.

What should we do if there are multiple file replaces?
@stamblerre stamblerre added this to the gopls/workspace-module milestone Feb 28, 2021
@gopherbot gopherbot added Tools gopls labels Feb 28, 2021
@stamblerre stamblerre changed the title x/tools/gopls: prefer higher versions when encountering conflicts in workspace module x/tools/gopls: handle conflicting replaces in the workspace module Feb 28, 2021
@stamblerre stamblerre added this to To Do in gopls on-deck Feb 28, 2021
@suzmue suzmue added the gopls/workspace label May 10, 2022
@findleyr
Copy link
Contributor

findleyr commented May 18, 2022

I don't think we're going to do this, since the workspace module is deprecated. go.work files have their own handling for this.

@findleyr findleyr closed this as not planned Jul 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
gopls/workspace gopls Tools
Projects
No open projects
gopls on-deck
To Do
Milestone
gopls/later
Development

No branches or pull requests
4 participants
@stamblerre @suzmue @gopherbot @findleyr