Wilken Rivera
5a87760702
makefile: Update code path for golangci-lint ( #8751 )
...
While `golangci-lint run` is the same as `golangci-lint run ./...` running it without a
path seems to throw warning messages related to its cache, which is confusing.
This change sets an explicit path for golangci-lint when calling `make lint` or `make ci-lint`.
2020-02-17 11:24:08 +01:00
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