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

chore: lint tests in modules other than gamm #5207

Closed
wants to merge 17 commits into from

Conversation

faddat
Copy link
Member

@faddat faddat commented May 17, 2023

What is the purpose of the change

Works toward closing, but does not close #5061

This PR is to keep momentum going without blocking on some type assertions in x/gamm. It lints all modules other
than x/gamm. Will continue on with x/gamm in #5127.

Testing and Verifying

If this passes the test suite, then it works because it focuses on linting tests.

Documentation and Release Note

No changelog because there's no change to the application.

Where is the change documented?

  • Specification (x/{module}/README.md)
  • Osmosis documentation site
  • Code comments?
  • N/A

@faddat faddat added V:state/compatible/no_backport State machine compatible PR, depends on prior breaks no-changelog labels May 17, 2023
@faddat faddat mentioned this pull request May 17, 2023
@faddat
Copy link
Member Author

faddat commented May 17, 2023

@mattverse -- fyi, I've tried "jiggling" the changelog reminder but to no avail. I think we are g2g here, and that #5127 can be looked at again after #5140.

Oh noes the tests just failed :/ . Moment...

@faddat
Copy link
Member Author

faddat commented May 17, 2023

@czarcas7ic should I use a different label or add a changelog?

<3

tests passing

@github-actions github-actions bot added C:x/gamm Changes, features and bugs related to the gamm module. C:x/twap Changes to the twap module labels May 18, 2023
@faddat faddat marked this pull request as ready for review May 18, 2023 17:34
@faddat
Copy link
Member Author

faddat commented May 18, 2023

Note: I really need to figure out what I am doing that breaks this... tomorrow

@faddat faddat marked this pull request as draft May 18, 2023 18:40
@faddat
Copy link
Member Author

faddat commented May 19, 2023

Closing in favor of #5127

@faddat faddat closed this May 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A:no-changelog C:app-wiring Changes to the app folder C:CLI C:x/concentrated-liquidity C:x/epochs C:x/gamm Changes, features and bugs related to the gamm module. C:x/incentives C:x/pool-incentives C:x/poolmanager C:x/tokenfactory C:x/twap Changes to the twap module T:CI V:state/compatible/no_backport State machine compatible PR, depends on prior breaks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

lint tests
4 participants