Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BITBUCKET_COMMENT_REPORTER comments on every CI/CD job even though it's unused. #3317

Closed
RelativeSure opened this issue Jan 24, 2024 · 5 comments · Fixed by #3321
Closed
Labels
bug Something isn't working

Comments

@RelativeSure
Copy link

Describe the bug
A clear and concise description of what the bug is.

In the new v7.8.0, the new BITBUCKET_COMMENT_REPORTER reporter has been released with the reporter being default set to true.
It comments in the workflow that it hasn't been setup. [Bitbucket Comment Reporter] Required Bitbucket CI CD variables not found, so skipped post of PR comment, this does not happen for any other reporter.

Can this please get removed or do we need to explicit set this to false so we don't get this message in our CI/CD?

To Reproduce
Steps to reproduce the behavior:

  1. Go to '…'
  2. Click on '….'
  3. Scroll down to '….'
  4. See error

Expected behavior
A clear and concise description of what you expected to happen.

Screenshots
If applicable, add screenshots to help explain your problem.

Additional context
Add any other context about the problem here.

@RelativeSure RelativeSure added the bug Something isn't working label Jan 24, 2024
@echoix
Copy link
Collaborator

echoix commented Jan 24, 2024

It prints this when running in bitbucket or another?

@RelativeSure
Copy link
Author

RelativeSure commented Jan 24, 2024

It prints this when running in bitbucket or another?

We run it towards a GitHub Enterprise where we use the GitHub Comment Reporter.
We run the megalinter-terraform flavor on Jenkins using the GitHub plugin.

We don't and have never used Bitbucket.

@nvuillam
Copy link
Member

PR on the way to remove useless logs :)

nvuillam added a commit that referenced this issue Jan 25, 2024
…ariable (#3321)

* Activate CI servers reporters only if we find a related default env variable

Fixes #3317

* [MegaLinter] Apply linters fixes

---------

Co-authored-by: nvuillam <[email protected]>
Co-authored-by: nvuillam <[email protected]>
@RelativeSure
Copy link
Author

Hi again,
Is this fix in v7.8.0 or is it first in v7.9.0 / v8.x.x?

Thanks in advance.

@echoix
Copy link
Collaborator

echoix commented Jan 29, 2024

Next release, 7.9, or now through the beta

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants