-
Notifications
You must be signed in to change notification settings - Fork 693
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
gofumpt for v4 #1511
gofumpt for v4 #1511
Conversation
Codecov Report
@@ Coverage Diff @@
## release/v4.2.x #1511 +/- ##
================================================
Coverage ? 0.41%
================================================
Files ? 7
Lines ? 483
Branches ? 0
================================================
Hits ? 2
Misses ? 480
Partials ? 1 |
Got to guess that the liveness check here no longer works -- thoughts? |
Seems not only that is needed to fix it. if I had to guess it is dependent on environment. |
I just got rid of the liveness test. |
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.
utACK
@@ -121,27 +121,3 @@ jobs: | |||
run: | | |||
make test-sim-multi-seed-short | |||
if: "env.GIT_DIFF != ''" | |||
|
|||
test-liveness: |
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.
was this failing?
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.
yes, but I think it's failing for process reasons?
Basically, it is an attempt to backport and the original v4 branch had a different test. I could take the .github folder from v4 and get rid of the diff, though.
Ah, I remember why it's failing now-- pretty sure it couldn't find a file in the environment.
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.
Another way to say this: v4 ci isn't maintained.
same as per comment from #1509:
|
Except that legibility is a security update. |
Description
Applies gofumpt to gaia v4, standardizing the codebase
#1449
For contributor use:
docs/
) or specification (x/<module>/spec/
)godoc
comments.Unreleased
section inCHANGELOG.md
Files changed
in the Github PR explorerFor admin use:
WIP
,R4R
,docs
, etc)