Skip to content

add option to exclude files from linting with super-linter - #72

Merged
juleshaas merged 3 commits into
mainfrom
exclude_option_for_json_linting
Mar 26, 2026
Merged

juleshaas merged 3 commits into
mainfrom
exclude_option_for_json_linting

Conversation

@juleshaas

@juleshaas juleshaas commented Mar 24, 2026 •

Copy link
Copy Markdown
Contributor

This MR adds the option to exlcude files from linting with super-linter in repos, where the reusable workflow is used.

To exclude files from linting with super-linter add them to the linting.yml in the corresponding repo, like:

jobs:
  lint:
    uses: mundialis/github-workflows/.github/workflows/linting.yml@main
    with:
      FILTER_REGEX_EXCLUDE: ".*processing/templates/template_MAIN_loop.json"

Attention: This skips the whole file during linting with super-linter! Other linters still apply.

@mmacata mmacata left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Useful feature, thank you!
I added some suggestions to make it more obvious that no python linting is affected or python file could be excluded and also to prefer more specific excludes if possible.

Comment thread .github/workflows/linting.yml Resolved
Comment thread .github/workflows/linting.yml Resolved
Comment thread README.md Resolved
Comment thread README.md Resolved
Comment thread README.md Resolved
Apply suggestions from code review

Co-authored-by: Carmen Tawalika <mmacata@users.noreply.github.com>
@juleshaas juleshaas changed the title add option to exclude files from linting add option to exclude files from linting with super-linter Mar 26, 2026
@juleshaas

Copy link
Copy Markdown
Contributor Author

Good idea, thanks. I added the suggestions and tested again successfully.

@juleshaas
juleshaas merged commit 7278286 into main Mar 26, 2026
3 checks passed
@juleshaas
juleshaas deleted the exclude_option_for_json_linting branch March 26, 2026 15:47
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.

2 participants