Commit ce3955da authored by Markus Bischof's avatar Markus Bischof
Browse files

fix(mega_linter): remove non-mandatory MegaLinter configuration variables 

Since environment variables has higher precedence than settings from the MegaLinter configuration file, it's not possible to use MegaLinter locally with the same configuration without having things configured in multiple files. E.g. it's not possible to define `DISABLE_LINTERS` in `.mega-linter.yml` as long as it's set via environment variables.

BREAKING-CHANGE: remove job variables `FILTER_REGEX_EXCLUDE`, `VALIDATE_ALL_CODEBASE` and `DISABLE_LINTERS`
parent 85461b23

jobs/mega_linter/CHANGELOG.md

+3 −0
Original line number Diff line number Diff line 
# Changelog

All notable changes to this job will be documented in this file.



## [3.0.0] - 2023-08-30

* Remove job variables `FILTER_REGEX_EXCLUDE`, `VALIDATE_ALL_CODEBASE` and `DISABLE_LINTERS` to make them configurable via MegaLinter configuration file



## [2.0.0] - 2023-03-07

* Bump megalinter image version to v6.20.0

* Bump sarif-codeclimate version to 2.1.2

jobs/mega_linter/mega_linter.yml

+0 −7
Original line number Diff line number Diff line
@@ -16,22 +16,15 @@ mega_linter: 
    ### MegaLinter variables

    DEFAULT_WORKSPACE: "${CI_PROJECT_DIR}"

    DEFAULT_BRANCH: "${CI_DEFAULT_BRANCH}"

    # Exclude files based on a regex

    FILTER_REGEX_EXCLUDE: ""

    # Directory in which your linters configuration are

    LINTER_RULES_PATH: ".linters"

    # Your mega linter config file if any

    MEGALINTER_CONFIG: ".mega-linter.yml"

    # If set to false will only scan diff files from previous commit

    VALIDATE_ALL_CODEBASE: "true"

    # Directory in which every report is filed

    REPORT_OUTPUT_FOLDER: "megalinter-reports"

    # Posts Mega-Linter results summary in the comments of the related merge request (if existing)

    # https://oxsecurity.github.io/megalinter/latest/reporters/GitlabCommentReporter/

    GITLAB_COMMENT_REPORTER: "true"

    # List of disabled linters in every case, this linter is deactivated due to

    # its time of processing (~10 minutes on tested projects)

    DISABLE_LINTERS: SPELL_CSPELL

    # Activate the SARIF reporter

    SARIF_REPORTER: "true"

    ### Custom variables