-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
x/auth: remove alias.go usage #6440
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK
Codecov Report
@@ Coverage Diff @@
## master #6440 +/- ##
=======================================
Coverage 55.98% 55.98%
=======================================
Files 466 466
Lines 27908 27908
=======================================
Hits 15623 15623
Misses 11180 11180
Partials 1105 1105 |
As a side note, the integrations tests are passing in github but locally they are failing. How can someone make sure that they are passing locally? |
hmm this is odd. I tested out locally and they passed. Im not sure if its a env thing but this is a bit concerning |
* x/auth: remove alias.go usage * Fix simd_test.go and formatting * Fix app.go formatting * Fix cli_test.go * Fix lint warnings * Undo accidental deletion during merge Co-authored-by: colin axner <[email protected]> Co-authored-by: Federico Kunze <[email protected]>
Description
This PR closes a sub task in #6311
Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.
docs/
) or specification (x/<module>/spec/
)godoc
comments.Unreleased
section inCHANGELOG.md
Files changed
in the Github PR explorer