oxsecurity / megalinter

🦙 MegaLinter analyzes 50 languages, 22 formats, 21 tooling formats, excessive copy-pastes, spelling mistakes and security issues in your repository sources with a GitHub Action, other CI tools or locally.
https://megalinter.io
GNU Affero General Public License v3.0
1.81k stars 215 forks source link

Megalinter with SPELL_LYCHEE disabled actually runs it #3076

Closed fchastanet closed 8 months ago

fchastanet commented 8 months ago

Describe the bug I run megalinter v7.5.0, I disabled SPELL_LYCHEE but it is actually running

To Reproduce I'm using this .mega-linter.yml file

---
# Configuration file for MegaLinter
# See all available variables at https://megalinter.io/latest/config-file/ and in linters documentation

APPLY_FIXES: all # all, none, or list of linter keys
# ENABLE: # If you use ENABLE variable, all other languages/formats/tooling-formats will be disabled by default
# ENABLE_LINTERS: # If you use ENABLE_LINTERS variable, all other linters will be disabled by default
DISABLE:
  - SQL
DISABLE_LINTERS:
  - JAVASCRIPT_ES
  - JAVASCRIPT_PRETTIER
  - RUBY_RUBOCOP
  - REPOSITORY_CHECKOV
  - REPOSITORY_TRIVY
  - REPOSITORY_KICS
  - SPELL_VALE
  - SPELL_LYCHEE
  - SPELL_PROSELINT
# DISABLE_ERRORS: true # Uncomment if you want MegaLinter to detect errors but not block CI to pass
EXCLUDED_DIRECTORIES:
  - '.history'
  - '.git'
  - '.idea'
  - 'logs'
FILTER_REGEX_EXCLUDE: |
  (?x)(
    \.git/|
    \.history/|
    \.idea/|
    ^vendor/
  )
FILEIO_REPORTER: false
PRE_COMMANDS:
  - command: env
    cwd: 'workspace'
POST_COMMANDS:
  # FIX files set as root user
  # HOST_USER_ID and HOST_GROUP_ID set in package.json
  - command: |
      if [[ "{HOST_USER_ID:-0}" != "0" && "{HOST_GROUP_ID:-0}" != "0" ]]; then
        find . -user 0 -exec chown ${HOST_USER_ID}:${HOST_GROUP_ID} {} ';'
      fi
    cwd: 'workspace'

  # remove files generated by cspell
  - command: find . -name '*megalinter_file_names_cspell.txt' -delete
    cwd: 'workspace'

PRINT_ALPACA: false
SHOW_ELAPSED_TIME: true

# Linters configurations
BASH_SHELLCHECK_FILTER_REGEX_EXCLUDE: |
  (?x)(
    ^vendor/.*|
    /testsData/
  )
BASH_SHELLCHECK_ARGUMENTS: --source-path=/tmp/lint
BASH_SHFMT_ARGUMENTS: -i 2 -ci

CREDENTIALS_SECRETLINT_CONFIG_FILE: .secretlintrc.yml

EDITORCONFIG_EDITORCONFIG_CHECKER_FILTER_REGEX_EXCLUDE: |
  (?x)(
    \.git/|
    ^Gemfile.lock$|
    ^.*-megalinter_file_names_cspell.txt
  )

GIT_GIT_DIFF_PRE_COMMANDS:
  - command: git config --global core.autocrlf input
    continue_if_failed: false
  - command: git config --global core.eol lf
    continue_if_failed: false
  - command: git config --global core.whitespace cr-at-eol,-trailing-space
    continue_if_failed: false
  - command: git config --global core.excludesfile .gitignore
    continue_if_failed: false
  - command: git config --global --add safe.directory .git
    continue_if_failed: false

IGNORE_GITIGNORED_FILES: true
IGNORE_GENERATED_FILES: true

JAVASCRIPT_DEFAULT_STYLE: prettier
JAVASCRIPT_ES_CONFIG_FILE: .eslintrc.js
JAVASCRIPT_ES_FILTER_REGEX_EXCLUDE: report/

JSON_JSONLINT_FILTER_REGEX_EXCLUDE: |
  (?x)(
    ^\.vscode/settings\.json|
    ^conf/\.vscode/settings\.json|
    ^\.vscode/launch\.json
  )
MARKDOWN_MARKDOWNLINT_FILTER_REGEX_EXCLUDE: |
  (?x)(
    ^pages/Commands.md|
    ^pages/FrameworkFullDoc.md|
    ^pages/README.md|
    ^pages/doc|
    ^pages/bashDoc|
    ^pages/tests
  )
SPELL_CSPELL_FILTER_REGEX_EXCLUDE: |
  (?x)(
    \.git/|
    ^\.history/|
    ^.*-megalinter_file_names_cspell.txt|
    ^megalinter-reports/|
    ^commit-msg.md$|
    \.svg$
  )
SPELL_FILTER_REGEX_EXCLUDE: |
  (?x)(
    \.git/|
    \.history/|
    \.svg$
  )

I'm running this command line

docker run --rm --name=megalinter -e HOST_USER_ID=1000 -e HOST_GROUP_ID=1000 -e MEGALINTER_CONFIG=.mega-linter-light.yml -e CLEAR_REPORT_FOLDER=true --workdir /tmp/lint -e APPLY_FIXES=all -it -v /var/run/docker.sock:/var/run/docker.sock:rw -v /home/wsl/fchastanet/my-notes2:/tmp/lint:rw oxsecurity/megalinter-terraform:v7.5.0

Here the megalinter.log

Expected behavior the SPELL_LYCHEE linter shouldn't be run as I explicitly disabled it

Screenshots N/A

Additional context You can reproduce it using this repository https://github.com/fchastanet/my-notes2

fchastanet commented 8 months ago

Sorry I found the issue wrong megalinter file was set

nvuillam commented 8 months ago

Glad your config is now ok :)