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

remove cosmossdk.io/x/* dependence in the root go module #18290

Closed
4 tasks
Tracked by #11899
tac0turtle opened this issue Oct 28, 2023 · 4 comments
Closed
4 tasks
Tracked by #11899

remove cosmossdk.io/x/* dependence in the root go module #18290

tac0turtle opened this issue Oct 28, 2023 · 4 comments
Assignees
Labels
dependencies Pull requests that update a dependency file T: Tests

Comments

@tac0turtle
Copy link
Member

tac0turtle commented Oct 28, 2023

While spinning out modules from the cosmos sdk go.mod we introduced module imports into the cosmos sdk. We need to remove these.

  • Baseapp
  • Client (marko)
  • Server
  • Testutils
@github-actions github-actions bot added the needs-triage Issue that needs to be triaged label Oct 28, 2023
@tac0turtle tac0turtle changed the title cleanup all module imports in cosmos-sdk remove cosmossdk.io/x/* dependence in the root go module Oct 28, 2023
@tac0turtle tac0turtle added T: Tests dependencies Pull requests that update a dependency file T:Sprint and removed needs-triage Issue that needs to be triaged labels Oct 28, 2023
@github-project-automation github-project-automation bot moved this to 👀 To Do in Cosmos-SDK Oct 28, 2023
@odeke-em odeke-em self-assigned this Oct 30, 2023
@odeke-em
Copy link
Collaborator

odeke-em commented Nov 8, 2023

Perhaps we should think about spinning out the Go module for "cosmossdk.io/testutil" because in the root go.mod, the reason why we need a bunch of those modules added to the top level is because testutil/network uses them, it doesn't have a go.mod file and can't use a replace directive and guess what? Almost every module imports "github.com/cosmos/cosmos-sdk/testuti/*" per this command

$ git grep 'github.com/cosmos/cosmos-sdk/testutil' | cut -d":" -f1 | xargs dirname | sort | uniq

and that currently returns unique 148 directories that import it

@tac0turtle
Copy link
Member Author

i dont think we should move the entire thing but parts of it, we will handle this issue

@tac0turtle
Copy link
Member Author

i think testutils is the only worry here. since baseapp client and server will be deleted this year

@tac0turtle
Copy link
Member Author

closing this in favour of #20279

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file T: Tests
Projects
Status: 🥳 Done
7 participants