Fix Value for container-name of type String required

[AlbanAndrieu]

Fixes #

Fix Value for container-name of type String required when running pre-commit run megalinter-incremental

Proposed Changes

1. ...
2. ...
3. ...

Readiness Checklist

Author/Contributor

• Add entry to the CHANGELOG listing the change and linking to the corresponding issue (if appropriate)
• If documentation is needed for this change, has that been included in this pull request

Reviewing Maintainer

• Label as breaking if this is a large fundamental change
• Label as either automation, bug, documentation, enhancement, infrastructure, or performance

[nvuillam]

I'm a newbie with pre-commit... :D
@Kurt-von-Laven , your opinion ? :)

[AlbanAndrieu]

It seems working for me on Ubuntu LTS with node v18.12.1.
I tested it on node, PHP, java, python and devops project.

If some other want workaround :

    rev: v6.15.0
    hooks:
      - id: megalinter-incremental # Faster, less thorough
        args:
          - mega-linter-runner
          - --containername
          - "megalinter-incremental"
          - --remove-container
          - --fix
          - --env
          - "'APPLY_FIXES=all'"
          - --env
          - "'CLEAR_REPORT_FOLDER=true'"
          - --env
          - "'LOG_LEVEL=warning'"
          - --filesonly
        stages:
          - commit

[nvuillam]

Let's have a leap of faith, you seem to know what you are doing :)

[AlbanAndrieu]

@nvuillam Thanks budy

[Kurt-von-Laven]

This looks fine, although the quotes don't do anything. Did you encounter some issue using the original pre-commit hook? It's possible there are some differences in behavior between platforms.