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

remove duplicated checks #85

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

remove duplicated checks #85

wants to merge 3 commits into from

Conversation

arxdsilva
Copy link

In #56 it's been said that the CI checks performance should be better for the time that It takes, so I checked all linters and removed possible duplicates, this way we should get a lower time in CI.

@tmc
Copy link
Owner

tmc commented Oct 29, 2018

Thanks for your contribution. Can you look I to the circle issue(s)? Thanks!

@arxdsilva
Copy link
Author

arxdsilva commented Oct 29, 2018

Hi @tmc , didnt find a link between this PR and the CIs break, btw there are a few things that can be added into gometalinter, such as --fast and --concurrency=X flags. The first only allows fast linters and the second runs X linters concurrently.

@tmc
Copy link
Owner

tmc commented Feb 6, 2019

Can you address this and re-run ci?

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.

2 participants