Commit graph

806 commits

Author SHA1 Message Date
Antonio Yang
60e2561b15
Add clippy for rust (#1277)
* Add clippy for Rust

* Update ARM test cases

- fix arm good test case
- update tap report

* Lint github workflows release config
2021-03-08 14:13:04 -06:00
Gabriel Diaz
f58e607c4b
Fix buggy diff cmd (#1314)
* Fix buggy diff cmd

* Fix new line for diff cmd output

Co-authored-by: Lukas Gravley <admiralawkbar@github.com>
2021-03-05 11:25:31 -06:00
Kim SeungSu
68c8bf9f11
fix, checkout DEFAULT_BRANCH for diff base (#1308) 2021-03-05 08:57:16 -06:00
Gabriel Diaz
11720172cb
Fix get file diff on pr event (#1305)
* Fix pr event sha

* Improve cmd

* Fix space
2021-03-04 13:51:44 -06:00
Gabriel Diaz
46df94844c
Add debug info for multi status api calls (#1287)
* Add debug info for multi status api calls

* Add test for clean action

* Add more verbose

* Test in action context

* Test use local build as action

* Improve debug for raw file array

* Fix shallow fetch

* Fix error messages on diff cmd

* Changed repeated code to function

* Fix quotes
2021-03-02 08:29:12 -06:00
Max Held
0b756c57e8
only error on rstats lintr errors, not all lints (#1233)
* only error on lintr errors

* ensure that dockerfile includes purrr

* update tap snapshot

* add actual lint error

* add snapshot for actual error

Co-authored-by: Gabriel Diaz <gabo.fdc@gmail.com>
Co-authored-by: Lukas Gravley <admiralawkbar@github.com>
2021-02-26 08:49:08 -06:00
Antonio Yang
f47b363f71
Add rustfmt for Rust (#1250)
* Add linter for Rust

- add 2015 edition
- add 2018 edition

* Add tap reports

- add tap reports
- fix good test case

* 1 command=lesslayers

Co-authored-by: Lukas Gravley <admiralawkbar@github.com>
2021-02-26 08:47:21 -06:00
Gabriel Diaz
71d36c41c2
Add logic to check command output before maping the files (#1259) 2021-02-25 18:15:51 +00:00
dependabot[bot]
7c4842d2d2
Bump ansible-lint from 5.0.1 to 5.0.2 in /dependencies (#1248)
* Bump ansible-lint from 5.0.1 to 5.0.2 in /dependencies

Bumps [ansible-lint](https://github.com/ansible-community/ansible-lint) from 5.0.1 to 5.0.2.
- [Release notes](https://github.com/ansible-community/ansible-lint/releases)
- [Commits](https://github.com/ansible-community/ansible-lint/compare/v5.0.1...v5.0.2)

Signed-off-by: dependabot[bot] <support@github.com>

* fix ansible

* fix ansible

* looking

* maybe baby

* adding details

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Admiral Awkbar <admiralawkbar@github.com>
2021-02-24 07:45:39 -06:00
Eric Nemchik
389fe11f9b
Use git ls-tree to build file list (#1192)
* Ignore files that are ignored by Git (#1185)

* Ignore files that are ignored by Git

* Add missing continue statement

* Fix linting errors

* Fix linting errors

* like it

* like it

* Add the IGNORE_GITIGNORED_FILES variable

* Fix setting the Ansible directory when running tests

* Fix var export

* Update linter.sh

Co-authored-by: Admiral Awkbar <admiralawkbar@github.com>

* Remove RenameTestFolder

* Update action.yml

* removing the dockignore from our test...

* fixed it

Co-authored-by: Marco Ferrari <ferrari.marco@gmail.com>
Co-authored-by: Admiral Awkbar <admiralawkbar@github.com>
2021-02-23 11:51:34 -06:00
Lukas Gravley
9bfc6158b9
Fix Tap report spacing (#1240)
* cleaner

* fix go

* fix go

* going ham

* fix typo

* fix typo

* fix typo

* fix mapfile

* fix mapfile

* fixed spacing
2021-02-22 11:28:49 -06:00
Marco Ferrari
b6bbc88e8e
Ignore files that are ignored by Git (#1185)
* Ignore files that are ignored by Git

* Add missing continue statement

* Fix linting errors

* Fix linting errors

* like it

* like it

* Add the IGNORE_GITIGNORED_FILES variable

* Fix setting the Ansible directory when running tests

* Fix var export

* Update linter.sh

Co-authored-by: Admiral Awkbar <admiralawkbar@github.com>
2021-02-18 12:15:50 -06:00
dependabot[bot]
570a113901
Bump cljkondo/clj-kondo from 2021.01.20-alpine to 2021.02.13-alpine (#1213)
* Bump cljkondo/clj-kondo from 2021.01.20-alpine to 2021.02.13-alpine

Bumps cljkondo/clj-kondo from 2021.01.20-alpine to 2021.02.13-alpine.

Signed-off-by: dependabot[bot] <support@github.com>

* fix ascii

* fix spacing

* removing non ascii chars

* fixed typo

* fixed typo

* add location

* flaky go

* flaky go

* flaky ansible

* fix groovy

* update

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Admiral Awkbar <admiralawkbar@github.com>
2021-02-18 09:59:43 -06:00
Jeremiah
00c70f4b2c
Update snakefmt to 0.3.0 and black to 20.3 (now supported by snakefmt). (#1212)
* Update snakefmt to 0.3.0 and black to 20.3 (now suppored by snakefmt).

* Update pipfile for ansible 5.0

* Align pipfile python with Docker alpine image

* Explicitly pin python minor version in dockerfile

* Ansible updates

* Add missing cryptography dependencies

* Eliminate deprecated ansible syntax

* Update expected output for ansible

* Try to add regex to handle test run length

* Yaml doc syntax fixes

* Flailing attempt at tap wildcard

* Catch dynamic output from ansible-lint

* Update regex

* Fix regex statement

* Fix tap output

* Update go error message (issue from master branch)

* Fix regex

* Ansible strikes back

* Fix tap output

* Fix json tap

* Update r tap
2021-02-17 07:32:40 -06:00
Thorbjörn 'Puggan' Sundragon
3abe0fc5e6
Move FILTER_REGEX_INCLUDE and FILTER_REGEX_EXCLUDE test (#1022)
* Move FILTER_REGEX_INCLUDE AND FILTER_REGEX_EXCLUDE before checking filetype

* missing continue, and a lint error

Co-authored-by: Marco Ferrari <ferrari.marco@gmail.com>
2021-02-03 09:52:12 -06:00
Marco Ferrari
ab8780a58f
Install, configure, and run jscpd. Add missing tests (#1032)
Co-authored-by: Gabo <gabo.fdc@gmail.com>
Co-authored-by: Matt Desmond <beardofedu@github.com>
Co-authored-by: Lukas Gravley <admiralawkbar@github.com>
2021-01-27 20:47:34 +01:00
Ishan Goel
c9471d70eb
Linter command output on newline (#1146)
When lint fails, print command output on a newline surrounded by "------"

Co-authored-by: Lukas Gravley <admiralawkbar@github.com>
Co-authored-by: Gabriel Diaz <gabo.fdc@gmail.com>
2021-01-20 12:44:30 -06:00
Lukas Gravley
e11786a54c
Adding Flag for Possum (#1149)
* adding a bad feature

* fix hadolint
2021-01-19 11:03:14 -06:00
Gabriel Diaz
88645c0977
Move switch back to code branch (#1139)
Co-authored-by: Lukas Gravley <admiralawkbar@github.com>
2021-01-12 12:55:02 -06:00
Lukas Gravley
8c3fefc145
fix logic for docker (#1119)
* fix logic

* remove space
2021-01-04 14:25:33 -06:00
Aaron Sky
328b17bbc8
Improved ANSIBLE_DIRECTORY validation to support the repository root (#1111)
* Improved ANSIBLE_DIRECTORY validation to support the repository root

Fixes #1110, which prevents linting playbooks at the root of the repository from being supported.

* Trim whitespace

* Remove undesired extra validation condition

Addressing review feedback from @ ferrarimarco

* Avoid mutating ANSIBLE_DIRECTORY beyond the extent that was already in place

Co-authored-by: Lukas Gravley <admiralawkbar@github.com>
2021-01-04 13:38:46 -06:00
Lukas Gravley
193de39afd
Adding support for Prettier (#1118)
* adding it

* fix spaces

* vars were backwards

* remove prettier from function test
2021-01-04 11:07:57 -06:00
Gabriel Diaz
3ce39c07d9
Check OpenAPI yaml files (#1094)
Co-authored-by: Lukas Gravley <admiralawkbar@github.com>
2020-12-23 18:16:17 -05:00
Marco Ferrari
156024e231
Simplify linter and worker by generalizing ansible-lint (#1035)
Co-authored-by: Lukas Gravley <admiralawkbar@github.com>
2020-12-04 16:04:09 -06:00
Lukas Gravley
6f1a131b14
fixing logic for file find (#1049)
* fixing logic

* fix warn error

* fix files

* maybe?

* fix test name

* fix test
2020-12-03 16:17:55 -06:00
Gabriel Diaz
cf0e6a9704
Change log level from warn to debug (#1034)
* Change log level from warn to debug

* Move 'Linting x file' to be printed only if files are found

Co-authored-by: Lukas Gravley <admiralawkbar@github.com>
2020-11-30 11:19:17 -06:00
Cesar Rodriguez
0f9370c453
Fixes Terrascan to perform checks for all supported Terraform cloud providers (#1015)
* updates to Terrascan v1.2.0

* updates command to explicitly scan terraform for all supported providers

* add no cache

* specify Pipfile

* verbose

* remove system

* remove cache

* put it back

* put it back

* clear

* adding cython

* more libs

* add system

Co-authored-by: Lukas Gravley <admiralawkbar@github.com>
Co-authored-by: SVN Migrator <someone@somewhere.com>
2020-11-20 11:44:57 -06:00
Marco Ferrari
241f9f2cad
Harden testing (#982) 2020-11-20 09:12:31 -06:00
Lukas Gravley
cf19607c3c
Gherkin (#1011)
* adding gherkin

* adding to array

* fix space

* Update Dockerfile

* adding rules

* adding rules

Co-authored-by: SVN Migrator <someone@somewhere.com>
2020-11-19 09:54:20 -06:00
Barry Pollard
7a907a722c
Ignore more files from linting (#1003)
* Ignore node_modules and env folders

* Exclude media files

* Change env directory to separate dirs

* Remove env files

* Sort in alphabetical order

Co-authored-by: Lukas Gravley <admiralawkbar@github.com>
2020-11-18 13:50:07 -06:00
Marco Ferrari
3c1cecb7f8
Exclude __pycache__ and .pytest_cache from file list (#998)
Co-authored-by: Lukas Gravley <admiralawkbar@github.com>
2020-11-13 09:05:04 -06:00
Lukas Gravley
fd154e1d13
give more memory (#992) 2020-11-12 11:45:04 -06:00
Lukas Gravley
7943ce1f54
break out: (#990)
* break out:

* fix it

* fix source

* add them exports

* more path

* more path
2020-11-12 11:27:34 -06:00
Lucas Gravley
874f0d9ebc quoting powershell files 2020-11-10 11:23:15 -06:00
Lucas Gravley
543986deee skip files 2020-11-10 10:02:03 -06:00
Marco Ferrari
b7e5dfce01 Fix shfmt errors 2020-11-06 23:10:09 +01:00
Lucas Gravley
c243939985 fix space 2020-11-06 13:28:37 -06:00
Lucas Gravley
4eae14fbe2 fix tap 2020-11-06 13:12:17 -06:00
Lucas Gravley
69695de34d fix space 2020-11-06 08:35:06 -06:00
Lucas Gravley
95d85d4dc2 fix it 2020-11-06 08:26:15 -06:00
Marco Ferrari
7862654394 Fix editorconfig-checker errors 2020-11-05 23:21:41 +01:00
Marco Ferrari
0778feaf42 Fix dotdirs exclusion 2020-11-05 17:48:56 +01:00
Lucas Gravley
995e2fc6bf fix find 2020-11-04 12:52:07 -06:00
Marco Ferrari
301afc458a Fix command output echo text 2020-11-04 14:18:45 +01:00
Marco Ferrari
aca43d0206 Fix SC2128 2020-11-04 10:26:09 +01:00
Marco Ferrari
c0441b2844 Use the already-defined linter names array to print linter versions 2020-11-04 10:26:05 +01:00
Lucas Gravley
2c9f8b1aa2 fixing logic 2020-11-03 10:06:33 -06:00
Marco Ferrari
3911130038 Fail on errors while getting linter versions 2020-11-02 21:54:26 +01:00
Lukas Gravley
5a04bd9875
Merge pull request #927 from github/fix-secondary-rules-file
Don't override the secondary rules file path if available, don't reuse dirty values from previous iterations when loading rules files
2020-11-02 08:58:06 -06:00
Lucas Gravley
3c911cfe99 make fatal 2020-11-02 07:46:58 -06:00
Gabo
95ca6f0006 Fix shellchekc for linter versions 2020-10-30 20:43:54 -05:00
Gabo
cc5556514f Fix shellcheck 2020-10-30 20:27:20 -05:00
Gabo
f826ac73c5 Validation should be inside container 2020-10-30 19:57:35 -05:00
Gabo
291489d6d9 Fix logging level for automation scripts 2020-10-30 19:09:54 -05:00
Gabo
e4b007cd40 Move validation to build time 2020-10-30 15:06:12 -05:00
Gabo
87036c3987 Better pswh module validation 2020-10-30 14:16:21 -05:00
Lukas Gravley
7f15e7a441
Merge branch 'master' into fix-secondary-rules-file 2020-10-30 10:25:58 -05:00
Marco Ferrari
e2f45db57d Initialize the LANGUAGE_LINTER_RULES value to avoid using leftovers from the previous iteration 2020-10-30 12:47:09 +01:00
Marco Ferrari
888bc7524e Don't override the secondary rules file path, if available 2020-10-30 12:15:16 +01:00
Gabo
997cfdcb02 Add notice warn and error log levels 2020-10-28 19:08:01 -05:00
Lucas Gravley
dbffb46526 adding tests 2020-10-28 10:22:55 -05:00
Lukas Gravley
18915fe6a9
Merge pull request #905 from IvanKuchin/IvanKuchin-bugfix#900
bugfix: #900
2020-10-27 13:47:22 -05:00
IvanKuchin
98a0a194ad
Update buildFileList.sh
fix syntax issue to pass bash linter
2020-10-24 14:46:30 -04:00
Marco Ferrari
5be174d8d9 Fix FILE_STATUS for Dockerfiles 2020-10-24 16:13:16 +02:00
IvanKuchin
ab3f02fe26
bugfix: #900 2020-10-23 12:16:14 -04:00
Ahmad Nassri
0396b9189b
fix: correct env variable name for JSX command (#890)
fixes #889
2020-10-23 09:17:25 +02:00
Gabo
ae8b16dac5 Fix tap test for dynamic tests 2020-10-19 20:46:14 -05:00
Gabo
3772bb9a03 Fix test case workspace_path in build file list 2020-10-19 16:15:04 -05:00
Lukas Gravley
4053087930
Merge pull request #882 from wrux/master
Fixed typo in missing linter rule fatal error.
2020-10-19 14:47:06 -05:00
Callum
abb0901e2e
Fixed typo in missing linter rules. 2020-10-19 21:16:33 +02:00
Gabo
874475eb70 Fix test case workspace_path 2020-10-19 14:05:38 -05:00
Marco Ferrari
fcce194d34 Merge TestCodebase and LintCodebase
Change-Id: I9a044fefe96e400b8e3320842b53fe97af2a7b40
2020-10-17 16:12:55 +02:00
Marco Ferrari
4452bff800 Fix rules files for JAVASCRIPT_ES and TYPESCRIPT_ES when testing 2020-10-15 16:45:18 +02:00
Marco Ferrari
5e25d5dcae Print linter commands 2020-10-15 16:45:18 +02:00
Marco Ferrari
2a4c89e92f Export LANGUAGE_LINTER_RULES 2020-10-15 16:45:18 +02:00
Marco Ferrari
fdb14fc19c Print environment variables before running linters 2020-10-15 16:45:18 +02:00
Marco Ferrari
df79a85546 Print environment variables before running linters 2020-10-15 16:45:18 +02:00
Marco Ferrari
4507a8e056 Add a debug instructions to check LANGUAGE_LINTER_RULES 2020-10-15 16:45:18 +02:00
Marco Ferrari
dfaeaf317d Simplify rules files loading, plus fixes 2020-10-15 16:45:18 +02:00
Lucas Gravley
d9b3473867 add warning: 2020-10-14 10:31:55 -05:00
Lukas Gravley
ed588b80c1
Merge pull request #846 from github/tekton
Adding tekton language support
2020-10-13 16:37:11 -05:00
Lucas Gravley
8be7c6df85 remove cache folder 2020-10-13 14:17:40 -05:00
Lucas Gravley
7a775854e7 cleaner 2020-10-13 13:46:46 -05:00
Lucas Gravley
c3a1403fa0 fix tekton to not trigger kuber 2020-10-13 12:54:14 -05:00
Lucas Gravley
d8b13b0fd8 now with files 2020-10-13 10:21:23 -05:00
Marco Ferrari
c201c0f6d1 Dynamically invoke linters 2020-10-13 09:08:03 +02:00
Lukas Gravley
e8091a4ee9
Merge pull request #827 from PascalBourdier/ignore-kustomize-file
ignore kustomize file with kubeval
2020-10-12 10:36:45 -05:00
Lukas Gravley
6bde44e755
Merge pull request #824 from mshimokura/ecrc_file
editorconfig-checker - Use .ecrc Files
2020-10-12 10:24:00 -05:00
Michael Shimokura
a5cada70f8 Revert "rename the variable to EDITORCONFIG_CHECKER_FILE_NAME for clarity"
This reverts commit 23a463867d.
2020-10-09 09:13:37 -04:00
Michael Shimokura
23a463867d rename the variable to EDITORCONFIG_CHECKER_FILE_NAME for clarity 2020-10-09 09:02:12 -04:00
Pascal Bourdier
32a5945dd8 embedding the negation in the existing if 2020-10-09 11:53:34 +02:00
Pascal Bourdier
d975791bb9 ignore kustomize file with kubeval
currently, kubeval doesn't support kustomize file

so kubeval returns a false-positive about a missing `metadata` key:
```
2020-10-09 06:19:19 [ERROR ]   Found errors in [kubeval] linter!
2020-10-09 06:19:19 [ERROR ]   [ERR  - .../kustomization.yml: Missing 'metadata' key]
2020-10-09 06:19:19 [ERROR ]   Linter CMD:[kubeval --strict .../kustomization.yml]
```
2020-10-09 09:02:09 +02:00
Michael Shimokura
cd4798ea04 only assign EDITORCONFIG_FILE_NAME if not already assigned 2020-10-08 16:17:29 -04:00
Michael Shimokura
1312a3a3e4 use the .ecrc file 2020-10-08 15:27:47 -04:00
Tim Jones
a8746a7159 Add support for Terragrunt files 2020-10-08 18:18:22 +02:00
Marco Ferrari
a2b3938ea2 Dynamically load linter rules 2020-10-07 19:02:36 +02:00
Marco Ferrari
68e15659b8 Centralize file list building logic in buildFileList.sh 2020-10-07 16:36:49 +02:00
Marco Ferrari
351496fe72 Populate FILE_ARRAY_DOCKERFILE_HADOLINT 2020-10-03 12:19:33 +02:00
Marco Ferrari
d210896355 Set FILE_ARRAY_EDITORCONFIG 2020-10-03 00:33:01 +02:00
Marco Ferrari
e1b1c7d223 Dynamically initialize FILE_ARRAY variables 2020-10-02 23:02:47 +02:00