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

GitHub Super Linter: ignore test json, zip, sol files #1088

Merged
merged 2 commits into from Mar 6, 2022
Merged

Conversation

elopez
Copy link
Member

@elopez elopez commented Mar 5, 2022

These test files are a large part of the repository, and do not need to be
linted. The large number of files makes the enumeration slow when performed
by the Super Linter, which results in CI jobs taking considerably long.

This PR ignores said files in hopes to speed up the CI workflows.
It also switches the Super Linter image to the slim version, which should
be slightly faster to download.

Fixes: #1084

These test files are a large part of the repository, and do not need to be
linted. The large number of files makes the enumeration slow when performed
by the Super Linter, which results in CI jobs taking considerably long.

This PR ignores said files in hopes to speed up the CI workflows.
It also switches the Super Linter image to the slim version, which should
be slightly faster to download.

Fixes: #1084
@montyly montyly changed the base branch from master to dev March 6, 2022 12:46
@montyly montyly merged commit e3392dd into dev Mar 6, 2022
@montyly montyly deleted the slow-lint branch March 6, 2022 13:25
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.

Fix CI's pylint speed
2 participants