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

put the new linter configuration on the develop branch #561

Merged
merged 2 commits into from
Sep 15, 2023

Conversation

faddat
Copy link
Contributor

@faddat faddat commented Sep 15, 2023

1. Summary

Adds a new linter configuration

2.Type of change

golangci-lint run ./... --fix

3. Implementation details

golangci-lint run ./... --fix

@faddat
Copy link
Contributor Author

faddat commented Sep 15, 2023

Should be merged before #561

@codecov
Copy link

codecov bot commented Sep 15, 2023

Codecov Report

Merging #561 (1b87714) into develop (13bf586) will not change coverage.
Report is 1 commits behind head on develop.
The diff coverage is n/a.

❗ Current head 1b87714 differs from pull request most recent head 4ee87f9. Consider uploading reports for the commit 4ee87f9 to get more accurate results

Impacted file tree graph

@@           Coverage Diff            @@
##           develop     #561   +/-   ##
========================================
  Coverage    13.19%   13.19%           
========================================
  Files          243      243           
  Lines        62942    62942           
========================================
  Hits          8306     8306           
  Misses       53967    53967           
  Partials       669      669           
Flag Coverage Δ
unittests 13.19% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed Coverage Δ
app/ante.go 72.72% <ø> (ø)
app/app.go 59.91% <ø> (ø)
app/modules.go 100.00% <ø> (ø)
app/test_helpers.go 54.81% <ø> (ø)
app/upgrades.go 60.60% <ø> (ø)
cmd/quicksilverd/bulk_airdrop.go 9.76% <ø> (ø)
cmd/quicksilverd/genairdrop.go 9.48% <ø> (ø)
cmd/quicksilverd/main.go 0.00% <ø> (ø)
cmd/quicksilverd/root.go 58.90% <ø> (ø)
test/simulation/helpers.go 0.00% <ø> (ø)
... and 97 more

@faddat faddat requested a review from ThanhNhann as a code owner September 15, 2023 05:25
@faddat faddat merged commit a28e49a into develop Sep 15, 2023
@joe-bowman joe-bowman deleted the faddat/quick-lint-reduce-changeset branch December 27, 2023 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant