Skip to content

CICD: Fixing golangci-lint and github action#4483

Merged
algojohnlee merged 14 commits intoalgorand:masterfrom
algojack:golangci-lint-github-action-debugging
Aug 30, 2022
Merged

CICD: Fixing golangci-lint and github action#4483
algojohnlee merged 14 commits intoalgorand:masterfrom
algojack:golangci-lint-github-action-debugging

Conversation

@algojack
Copy link
Copy Markdown
Contributor

@algojack algojack commented Aug 29, 2022

  • fixing golangci-lint command
  • fixing github action failing if it fails to run
  • adding linters for tests per @cce
  • @cce fixing some linter errors from before
  • removing golangci-lint and gofmt from codegen_verification.sh since it runs in reviewdog

@algojack algojack self-assigned this Aug 29, 2022
@codecov
Copy link
Copy Markdown

codecov Bot commented Aug 29, 2022

Codecov Report

Merging #4483 (33533aa) into master (edce4c1) will increase coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4483      +/-   ##
==========================================
+ Coverage   55.21%   55.26%   +0.04%     
==========================================
  Files         398      398              
  Lines       50297    50297              
==========================================
+ Hits        27772    27797      +25     
+ Misses      20191    20169      -22     
+ Partials     2334     2331       -3     
Impacted Files Coverage Δ
catchup/service.go 69.62% <0.00%> (+0.24%) ⬆️
cmd/tealdbg/debugger.go 73.49% <0.00%> (+0.80%) ⬆️
ledger/tracker.go 74.89% <0.00%> (+0.85%) ⬆️
agreement/proposalManager.go 98.03% <0.00%> (+1.96%) ⬆️
agreement/cryptoVerifier.go 69.71% <0.00%> (+2.11%) ⬆️
network/wsPeer.go 68.21% <0.00%> (+2.73%) ⬆️
ledger/blockqueue.go 88.50% <0.00%> (+2.87%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Comment thread scripts/buildtools/versions Outdated
Comment thread scripts/travis/codegen_verification.sh Outdated
@algojack algojack changed the title Fixing golangci-lint and github action CICD: Fixing golangci-lint and github action Aug 29, 2022
@algojack algojack marked this pull request as ready for review August 29, 2022 21:34
@algojack algojack requested review from a team, algobarb, algolucky and egieseke August 30, 2022 14:28
Copy link
Copy Markdown
Contributor

@algobarb algobarb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

txn(t, l, eval, &inner)
vb := endBlock(t, l, eval)
innerId := vb.Block().Payset[0].ApplicationID
innerID := vb.Block().Payset[0].ApplicationID
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did the linter catch this?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes, it caught it, and cce fixed it

@algojohnlee algojohnlee merged commit e83acec into algorand:master Aug 30, 2022
@algojack algojack deleted the golangci-lint-github-action-debugging branch August 30, 2022 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants