Commit Graph

137 Commits

Author SHA1 Message Date
Wilken Rivera 9ec8b67392
Add golangci-lint to project (#8686)
* Add golangci-lint as linting tool

* Disable failing staticchecks to start; GitHub issue to handle coming soon

* Run `goimports -w` to repair all source files that have improperly
formatted imports

* makefile: Add ci-lint target to run on travis

This change adds a new make target for running golangci-lint on newly
added Go files only. This target is expected to run during Packer ci builds.

* .github/contributing: Add code linting instructions

* travis: Update job configuration to run parallel builds
2020-02-14 11:42:29 -05:00
Adrien Delorme 75a639775e Update Makefile
there's no find dir
2019-10-16 10:24:34 +02:00
Adrien Delorme 1158eeb46d Update Makefile
to @go install ./cmd/mapstructure-to-hcl2 so that we can `make generate` and tests that generated code is what it should be
2019-10-15 12:56:43 +02:00
Adrien Delorme 597a3e323b Merge remote-tracking branch 'origin/master' into scrape_doc_to_builder_struct_config 2019-09-10 13:20:07 +02:00
Adrien Delorme 1091066c57 go get use enumer@master 2019-09-06 12:15:48 +02:00
Adrien Delorme c77a0bd196 remove goimports dependency as the generated code is now correct and won't need fixing
this should save us from running this command every now an then
2019-09-06 12:15:27 +02:00
Adrien Delorme 22a89ccf8a fix comment typo 2019-09-05 17:19:50 +02:00
Adrien Delorme a2e934e7fb Makefile: document why 2019-09-05 17:17:04 +02:00
Adrien Delorme e4563655fd Makefile: go to temporary directory to avoid go get of binaries from changing go deps 2019-09-05 17:11:08 +02:00
Adrien Delorme 4cb7c30987 Merge remote-tracking branch 'origin/master' into scrape_doc_to_builder_struct_config 2019-08-21 16:44:23 +02:00
Megan Marsh b82ab2cc6a change installation of pigeon if off gopath 2019-07-01 12:16:50 -07:00
Adrien Delorme 4c1abf828c
Merge branch 'master' into scrape_doc_to_builder_struct_config 2019-06-19 11:21:06 +02:00
Megan Marsh 0a2f4d884a make bin currently won't work outside of GOPATH 2019-06-14 11:12:58 -07:00
Adrien Delorme 6fcf581e3e remove all generated doc files before generating it again
in order to remove files that should be
2019-06-14 14:58:58 +02:00
Adrien Delorme 4ab5471490 Makefile: install-gen-deps now installs ./cmd/struct-markdown 2019-06-06 17:34:17 +02:00
Adrien Delorme 36ee40b2a1 test that go generated code if what is should be 2019-05-27 16:00:20 +02:00
Adrien Delorme 5a0b5e3944 Makefile: don't run fmt-check in test/testrace
* this is now run on a different circle ci track
2019-05-27 15:42:58 +02:00
Adrien Delorme deb2212e9e Makefile: use `go fmt`
fmt-check no runs go fmt and then a git diff; this should also fail on line ending diffs
2019-05-27 15:37:03 +02:00
Adrien Delorme 768736425e Makefile: remove unused golang.org/x/tools/cmd/stringer dep 2019-05-27 15:07:49 +02:00
Adrien Delorme 26d0448142 Makefile: split install-build-deps vs install-gen-deps 2019-05-27 15:06:49 +02:00
Adrien Delorme abd6eda6c9 generate execution policy from makefile
* make generate now generates on `./...` instead of `.`
2019-05-27 14:52:04 +02:00
Adrien Delorme 267e2253a2 check if running `go mod vendor` has an effect on vendor directory and if so fail 2019-04-11 14:19:25 +02:00
Adrien Delorme 5a896367ef make sure CI builds with GO111MODULE=off 2019-04-11 14:19:25 +02:00
Megan Marsh 12bd1c5f96 increase timeout for tests to reduce flakiness in travis 2019-04-05 10:57:47 -07:00
Adrien Delorme fef25e0b24 Revert "remove vet which seems to have been broken in golang v1.12"
This reverts commit 58f0b8c22e.
2019-03-04 11:18:00 +01:00
Adrien Delorme b842adcffd Update Makefile
make sure we use "go list ./..."
* without removing vendor dir as it is now removed by default.
* stop using ls -d */
2019-03-04 11:17:58 +01:00
Adrien Delorme e9990e8876 use `go vet` instead of `go tool vet`
https://golang.org/doc/go1.12#vet
2019-03-04 11:17:58 +01:00
Megan Marsh 58f0b8c22e remove vet which seems to have been broken in golang v1.12 2019-02-28 14:51:29 -08:00
xinau f5b13e3cb5 added inspec.io provisioner 2019-01-20 15:43:47 +00:00
Megan Marsh f51707630d clean up config.yml and decrease number of parallel processes in build 2018-11-19 16:02:43 -08:00
Adrien Delorme 03620d15c2 Makefile: make find work on windows using `-executable` instead of `-perm +111` 2018-10-16 16:23:07 +02:00
Adrien Delorme 69d59a83c2 Makefile: remove unrequired deps from test 2018-10-16 15:29:01 +02:00
Adrien Delorme 401fddbd33 Makefile remove duplicate go get 2018-10-16 15:28:35 +02:00
Adrien Delorme 161eb23978 makefile: put .PHONY at the top ? 2018-10-16 14:59:16 +02:00
Adrien Delorme ac29ee59f8 makefile: ci now runs only testrace, default runs testrace 2018-10-16 12:08:52 +02:00
Adrien Delorme 8936ddaedf makefile: testrace now uses `deps fmt-check mode-check vet` 2018-10-16 12:05:05 +02:00
Adrien Delorme 389280a530 makefile: refactor vet into it's own command 2018-10-16 12:04:41 +02:00
Adrien Delorme 09e0f2b93b makefile: mode-check ignore ./website/tmp folder 2018-10-16 12:00:15 +02:00
Megan Marsh 268886b7d3 exclude git files (including git hooks) from the executable check 2018-09-06 10:23:58 -07:00
Matthew Hooker 725596268b
Whitelist website vendor from mode check 2018-08-31 11:35:58 -07:00
Matthew Hooker 62014840b4
remove sync from makefile since we commit vendor directory. 2018-08-24 20:52:47 -07:00
Rickard von Essen b8adf689fd Check that only certain files are executable 2018-08-15 16:08:50 +02:00
Megan Marsh 66c45273fb
Merge pull request #6494 from double16/vagrant-docker
Vagrant post-processor for using a Docker image
2018-07-23 12:18:52 -07:00
Patrick Double 4f9a91012f Change docker-push to return docker-import artifact 2018-07-20 15:27:29 -05:00
Matthew Hooker 9384f322f6
Prevent make fmt from failing in default case
Apologies for the terrible hack. This just makes sure we always have
a default file to format, so if the unformatted list is empty
we don't error.
2018-07-17 13:30:10 -07:00
jabbera 4662d3eb95 make deps should download goimports 2018-07-09 14:49:32 -04:00
Matthew Hooker fac6b2e71b
don't send every source file through the shell 2018-05-01 22:55:10 -07:00
Matthew Hooker c163fbed35
use xargs to check for formatting 2018-05-01 20:39:48 -07:00
Matthew Hooker 2fc67f2210
Revert "Merge pull request #4494 from tb3088/EOL-handling"
This reverts commit dfe4f56c75, reversing
changes made to e90c87f885.

Revert changes to makefile and format checker
2018-05-01 13:09:55 -07:00
M. Marsh dfe4f56c75
Merge pull request #4494 from tb3088/EOL-handling
Eol handling for the Repo and build process
2018-05-01 13:00:15 -07:00