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

CI fixes and improvements #780

Merged
merged 4 commits into from
Apr 4, 2019
Merged

Conversation

Helcaraxan
Copy link
Contributor

@Helcaraxan Helcaraxan commented Mar 31, 2019

Multiple small pieces in this PR:

  • Fixing the CI builds as it was broken due to the recent update of TravisCI config to Go 1.12.x
  • Moving around CI steps to push them into CI scripts (that can be run locally) and use a build matrix to test in different environments.
  • The algo parameter for the unparam linter has been removed and we
    should thus no longer specify it in our .golangci.yaml. Also, bumping the GolangCI-lint
    version to the latest available minor release.
    See:

The `algo` parameter for the `unparam` linter has been removed and we
should thus no longer specify it. Also, bumping the GolangCI-lint
version to the latest available minor release.

See: mvdan/unparam@e6a6d1c
@Helcaraxan Helcaraxan self-assigned this Mar 31, 2019
@Helcaraxan Helcaraxan force-pushed the update-lint-config branch 4 times, most recently from c3220f6 to 31fb03c Compare March 31, 2019 18:56
@Helcaraxan Helcaraxan changed the title Update GolangCI-lint and lint config CI fixes and improvements Mar 31, 2019
@42wim
Copy link
Owner

42wim commented Apr 4, 2019

looking good, thanks!

@42wim 42wim merged commit 7f0e4ad into 42wim:master Apr 4, 2019
@Helcaraxan Helcaraxan deleted the update-lint-config branch April 5, 2019 08:06
zeridon pushed a commit to zeridon/matterbridge that referenced this pull request Feb 12, 2020
* Update GolangCI-lint and lint config

The `algo` parameter for the `unparam` linter has been removed and we
should thus no longer specify it. Also, bumping the GolangCI-lint
version to the latest available minor release.

See: mvdan/unparam@e6a6d1c

* Fix and improve bintray CI script

* Further CI setup improvements

* Split-out CI steps into stand-alone scripts
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