REPOSITORY_SEMGREP_RULESETS throws a v8r schema error about megalinter HOT 7 CLOSED

@wesley-dean-flexion i'll add it when releasing, that's ok :)

from megalinter.

Thank you.

from megalinter.

https://github.com/oxsecurity/megalinter/blob/main/.automation/build.py#L2287-L2290 is where the JSON schema is generated when stuff is added

from megalinter.

Some parts of the json schema are manually added, and we forgot to add this variable, good catch :)

Would u like to make a PR ? :)

Meanwhile, you can use YAML_V8R_FILTER_REGEX_EXCLUDE

from megalinter.

make a PR?
Please!

Next, I started looking through the repo for instances of something that made it into the schema (_ARGUMENTS) and something that didn't (_RULESETS) to try to figure out where it could be added. Given your comment, I obviously wasn't successful with this approach. :)

[wdean@yaoguai ~/src/megalinter][04:38 PM] (main) $ ack -li REPOSITORY_SEMGREP_ARGUMENTS
docs/descriptors/repository_semgrep.md
docs/json-schemas/configuration.html
megalinter/descriptors/schemas/megalinter-configuration.jsonschema.json
megalinter/linters/SemgrepLinter.py
site/descriptors/repository_semgrep/index.html
site/json-schemas/configuration.html
site/search/search_index.json
[wdean@yaoguai ~/src/megalinter][04:38 PM] (main) $ ack -li REPOSITORY_SEMGREP_RULESET
CHANGELOG.md
docs/descriptors/repository_semgrep.md
megalinter/descriptors/repository.megalinter-descriptor.yml
megalinter/linters/SemgrepLinter.py
site/CHANGELOG/index.html
site/descriptors/repository_semgrep/index.html
site/search/search_index.json

(including this so if someone else should come behind me, they might see what did (or didn't) work)

from megalinter.

https://github.com/oxsecurity/megalinter/blob/main/megalinter/descriptors/schemas/megalinter-configuration.jsonschema.json#L14266

 "REPOSITORY_SEMGREP_ARGUMENTS": {
      "$id": "#/properties/REPOSITORY_SEMGREP_ARGUMENTS",
      "description": "REPOSITORY_SEMGREP: User custom arguments to add in linter CLI call",
      "examples:": [
        "--foo",
        "bar"
      ],
      "items": {
        "type": "string"
      },
      "title": "REPOSITORY_SEMGREP: Custom arguments",
      "type": [
        "array",
        "string"
      ]
    },

^ currently exists

 "REPOSITORY_SEMGREP_RULESETS": {
      "$id": "#/properties/REPOSITORY_SEMGREP_RULESETS",
      "description": "REPOSITORY_SEMGREP: Specify custom ruleset(s) for semgrep",
      "examples:": [
        "auto",
        "p/ci"
      ],
      "items": {
        "type": "string"
      },
      "title": "REPOSITORY_SEMGREP: Custom arguments",
      "type": [
        "array",
        "string"
      ]
    },

^ proposed addition

@nvuillam something like that?

from megalinter.

see #3164

I didn't add an entry to the changelog as the update was a trivial bug fix. I can add something if you want.

from megalinter.