summaryrefslogtreecommitdiff
path: root/hack/validate/default
Commit message (Collapse)AuthorAgeFilesLines
* hack/validate: drop changelog-related stepsBjorn Neergaard2022-11-151-2/+0
| | | | Signed-off-by: Bjorn Neergaard <bneergaard@mirantis.com>
* Revert "validation: temporarily allows changes in integration-cli"Sebastiaan van Stijn2022-09-101-1/+1
| | | | | | This reverts commit 7ed823ead91938e5cc8de7a42ff93a25abe73e7e. Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
* validate: address SC2155 (shellcheck)Sebastiaan van Stijn2022-08-291-1/+1
| | | | | | | | | see https://github.com/koalaman/shellcheck/wiki/SC2155 Looking at how these were used, I don't think we even need to export them, so removing that. Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
* validate: add yamllint validationSebastiaan van Stijn2022-08-291-0/+1
| | | | | | | | | | | | | | | | validate other YAML files, such as the ones used in the documentation, and GitHub actions workflows, to prevent issues such as; - 30295c1750714d26f3c8fc9c3451f11ac351f2be - 8e8d9a36500fb07fa9d1b68539756b9a93d3509e With this patch: hack/validate/yamllint Congratulations! yamllint config file formatted correctly Congratulations! YAML files are formatted correctly Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
* validation: temporarily allows changes in integration-cliCrazyMax2022-06-171-1/+1
| | | | Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
* hack/validate: skip DCO in "default"Sebastiaan van Stijn2021-05-171-1/+2
| | | | | | | | | | We perform a DCO check before we run all other tests, so we can skip it as part of the validate step. Leaving the line in for visibility, and in case we switch from Jenkins to (e.g.) GitHub actions. Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
* run shfmtAkihiro Suda2020-03-031-1/+1
| | | | | | git grep --name-only '^#!' | egrep -v '(vendor|\.go|Jenkinsfile)' | xargs shfmt -w -bn -ci -sr Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
* validate: add shfmtAkihiro Suda2020-03-031-0/+1
| | | | Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
* hack/validate: clean-up output, move golang-ci-lint to endSebastiaan van Stijn2019-12-191-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The golang-ci-lint takes longest to run, so do the other checks first; this also makes the output a bit more readable. While at it, fix some consistency issues in the output of the other checks. Before this change: TEST_FORCE_VALIDATE=1 hack/validate/default Congratulations! All commits are properly signed with the DCO! Congratulations! Seccomp profile generation is done correctly. INFO [config_reader] Used config file hack/validate/golangci-lint.yml INFO [lintersdb] Active 15 linters: [deadcode gofmt goimports golint gosec gosimple govet ineffassign misspell staticcheck structcheck typecheck unconvert unused varcheck] ... INFO Execution took 2m12.4287302s Congratulations! "./pkg/..." is safely isolated from internal code. The swagger spec at "api/swagger.yaml" is valid against swagger specification 2.0 Congratulations! All api changes are done the right way. Congratulations! All toml source files changed here have valid syntax. Congratulations! Changelog CHANGELOG.md is well-formed. Congratulations! Changelog CHANGELOG.md dates are in descending order. After this change: TEST_FORCE_VALIDATE=1 hack/validate/default Congratulations! All commits are properly signed with the DCO! Congratulations! Seccomp profile generation is done correctly. Congratulations! Packages in "./pkg/..." are safely isolated from internal code. Congratulations! The swagger spec at "api/swagger.yaml" is valid against swagger specification 2.0 Congratulations! All API changes are done the right way. Congratulations! All TOML source files changed here have valid syntax. Congratulations! Changelog CHANGELOG.md is well-formed. Congratulations! Changelog CHANGELOG.md dates are in descending order. Congratulations! No new tests were added to integration-cli. INFO Start validation with golang-ci-lint INFO [config_reader] Used config file hack/validate/golangci-lint.yml INFO [lintersdb] Active 15 linters: [deadcode gofmt goimports golint gosec gosimple govet ineffassign misspell staticcheck structcheck typecheck unconvert unused varcheck] ... INFO Execution took 2m12.4287302s Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
* hack/validate/default: fix minor linting issueSebastiaan van Stijn2019-12-191-10/+10
| | | | Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
* Revert "hack: temporarily disable deprecate-integration-cli"Sebastiaan van Stijn2019-12-191-1/+1
| | | | | | This reverts commit 5f67568bfd08a10f51a819cae8696f591881f26b. Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
* Revert "Revert "hack: temporarily disable deprecate-integration-cli""Sebastiaan van Stijn2019-09-181-1/+1
| | | | | | This reverts commit 01332b54e02895c68b0ea3e4e1ff9d5dd8a0302a. Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
* Replace gometalinter with golangci-lintSebastiaan van Stijn2019-09-181-1/+1
| | | | | | | Replacing gometalinter, because it has been deprecated, and the repository was archived. Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
* Revert "hack: temporarily disable deprecate-integration-cli"Sebastiaan van Stijn2019-09-101-1/+1
| | | | | | This reverts commit beadc92e0781a32854f4cbb8389804a06730c351. Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
* hack: temporarily disable deprecate-integration-cliTibor Vass2019-09-091-1/+1
| | | | Signed-off-by: Tibor Vass <tibor@docker.com>
* hack: update scriptsTibor Vass2019-09-091-1/+0
| | | | | | | | | - remove -check.* flags - use (per-test) -timeout flag - allow user to override TEST_SKIP_* regardless of TESTFLAGS - remove test-imports validation Signed-off-by: Tibor Vass <tibor@docker.com>
* Revert "validate: temporarily disable deprecate-integration-cli as part of a ↵Tibor Vass2019-06-191-1/+1
| | | | | | | | revert" This reverts commit 3f1cdd5364d87f9a4fa4f62428f4e4bccbb594e4. Signed-off-by: Tibor Vass <tibor@docker.com>
* validate: temporarily disable deprecate-integration-cli as part of a revertTibor Vass2019-06-181-1/+1
| | | | Signed-off-by: Tibor Vass <tibor@docker.com>
* Bash scripts; use double brackets, fix bare variables, add quotesSebastiaan van Stijn2018-12-241-11/+11
| | | | | | | | | | | These scripts explicitly use Bash, so we should be able to use `[[` instead of `[` (which seems to be recommended). Also added curly brackets to some bare variables, and quoted some paths. This makes my IDE a bit more silent :-) Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
* Merge pull request #34551 from dnephin/use-gometalinterYong Tang2017-08-191-3/+1
|\ | | | | Use gometalinter for linting
| * Use gometalinter for lintingDaniel Nephin2017-08-181-3/+1
| | | | | | | | Signed-off-by: Daniel Nephin <dnephin@docker.com>
* | Add a validation for integration-cli deprecation.Daniel Nephin2017-08-181-0/+1
|/ | | | Signed-off-by: Daniel Nephin <dnephin@docker.com>
* Remove compose-bindata script, it should move to docker/cliDaniel Nephin2017-05-091-1/+0
| | | | Signed-off-by: Daniel Nephin <dnephin@docker.com>
* Add compose-bindata validation to default.Daniel Nephin2017-03-081-0/+1
| | | | Signed-off-by: Daniel Nephin <dnephin@docker.com>
* Convert script shebangs from "#!/bin/bash" to "#!/usr/bin/env bash"Tianon Gravi2017-02-131-1/+1
| | | | | | This is especially important for distributions like NixOS where `/bin/bash` doesn't exist, or for MacOS users who've installed a newer version of Bash than the one that comes with their OS. Signed-off-by: Andrew "Tianon" Page <admwiggin@gmail.com>
* validate changelog dates are in descending orderAndrew Hsu2017-01-231-0/+1
| | | | Signed-off-by: Andrew Hsu <andrewhsu@docker.com>
* validate changelog formatAndrew Hsu2016-12-071-0/+1
| | | | Signed-off-by: Andrew Hsu <andrewhsu@docker.com>
* Add validate/swagger-genDaniel Nephin2016-11-091-0/+1
| | | | Signed-off-by: Daniel Nephin <dnephin@docker.com>
* Add swagger.yaml validationBen Firshman2016-11-071-0/+1
| | | | | | | - yamllint to ensure it is a valid YAML file - go-swagger validate to ensure it is a valid swagger file Signed-off-by: Ben Firshman <ben@firshman.co.uk>
* Move validation out of hack/makeDaniel Nephin2016-11-031-0/+14
Allow each script to run directly without the hack/make.sh wrapper. These scripts do not produce artifacts and do not benefit from the "bundles" framework. Signed-off-by: Daniel Nephin <dnephin@docker.com>
View Lorry Controller Status