gitlab comment reporter, misleading error message

[LBeaulaton]

For gitlab CI, if the access token is badly defined (see the correct way: https://github.com/oxsecurity/megalinter#gitlab-ci), the following line asks for defining GITLAB_MEGALINTER_ACCESS_TOKEN while it should be GITLAB_ACCESS_TOKEN_MEGALINTER

megalinter/megalinter/reporters/GitlabCommentReporter.py

Line 169 in 5e7d9e3

"GITLAB_MEGALINTER_ACCESS_TOKEN containing a personal token with scope 'api'\n"

Note also that in the readme you do not precise that the access token should have the appropriate role (when it is a project token and not a personal token like in the demo) to comment on a merge request (at least developer).

PS: thanks for your great job

[nvuillam]

@LBeaulaton good catch , thanks :)

Would you like to make a quick PR ? :)

[LBeaulaton]

never done this, I can try

[nvuillam]

@LBeaulaton contributing to open source is a great experience :) ( be careful, if you're like me, once you'll start you'll never stop :D )

https://megalinter.io/latest/contributing/#submitting-a-pull-request

(as this is a very simple PR you may just fork, update the code and create a PR, no need to locally build )