Skip to content

Remove double lint#3748

Merged
ebuchman merged 1 commit intotendermint:developfrom
tac0turtle:marko/remove_double-lint
Jun 24, 2019
Merged

Remove double lint#3748
ebuchman merged 1 commit intotendermint:developfrom
tac0turtle:marko/remove_double-lint

Conversation

@tac0turtle
Copy link
Contributor

  • Circel CI and Golangci were doing linting jobs, removed circleci

This cannot be merged until it is removed from circleci.

@mircea-c can you remove it from circleci?

Signed-off-by: Marko Baricevic marbar3778@yahoo.com

  • Updated all relevant documentation in docs
  • Updated all code comments where relevant
  • Wrote tests
  • Updated CHANGELOG_PENDING.md

- Circel CI and Golangci were doing linting jobs, removed circleci

Signed-off-by: Marko Baricevic <marbar3778@yahoo.com>
@codecov-io
Copy link

Codecov Report

Merging #3748 into develop will increase coverage by 0.22%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           develop   #3748      +/-   ##
==========================================
+ Coverage    63.97%   64.2%   +0.22%     
==========================================
  Files          242     242              
  Lines        20010   20010              
==========================================
+ Hits         12801   12847      +46     
+ Misses        6166    6125      -41     
+ Partials      1043    1038       -5
Impacted Files Coverage Δ
consensus/reactor.go 71.39% <0%> (-0.24%) ⬇️
p2p/pex/pex_reactor.go 82.05% <0%> (+1.17%) ⬆️
privval/signer_validator_endpoint.go 87.77% <0%> (+2.22%) ⬆️
blockchain/pool.go 86.84% <0%> (+6.57%) ⬆️
privval/socket_listeners.go 96.55% <0%> (+6.89%) ⬆️
blockchain/reactor.go 79.9% <0%> (+8.41%) ⬆️

Copy link
Contributor

@melekes melekes left a comment

Choose a reason for hiding this comment

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

👍

@mircea-c
Copy link

I tried to update this check manually, but it doesn't look like CircleCI is using the github checks api (possibly because it's still under active development)

To merge this you'll need a repo admin to disable the check from the branch protection rules:

https://circleci.com/docs/2.0/enable-checks/#github-checks-waiting-for-status-in-github

@ebuchman ebuchman merged commit a44c621 into tendermint:develop Jun 24, 2019
unclezoro pushed a commit to unclezoro/tendermint that referenced this pull request Sep 6, 2019
- Circel CI and Golangci were doing linting jobs, removed circleci

Signed-off-by: Marko Baricevic <marbar3778@yahoo.com>
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.

5 participants