search

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.9k stars 232 forks source link

[TAP REPORTER] configuration error #744

Closed llaville closed 2 years ago

llaville commented 3 years ago

Describe the bug The configuration detailed on page https://github.com/nvuillam/mega-linter/blob/master/docs/reporters/TapReporter.md#configuration does not match the current v4.45 implementation (https://github.com/nvuillam/mega-linter/blob/v4.45.0/megalinter/reporters/TapReporter.py#L21-L41)

To Reproduce Add nothing in .mega-linter.yml config file because, it's supposed to have TAP activated by default

Expected behavior A tap sub-folder in report directory with tap logs

Screenshots Nothing produced

Additional context Workaround is to set TEXT_REPORTER: true

llaville commented 3 years ago

I found another side effect of wrong implementation (using TEXT_REPORTER, rather than TAP_REPORTER)

When you want to deactivate TEXT_REPORTER for remote GA run and activate it for local run (debugging purpose), you have TAP reporter also generated locally.

Real Use Case may be found below (workaround for local run vs GA run)

nvuillam commented 3 years ago

Seems to be a typo, it should be TAP_REPORTER in the code :)