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

Pull requests: automatically hide/remove the "codescene analysis & co" reports #408

Open
AlexisDrogoul opened this issue Feb 8, 2025 · 2 comments
Assignees
Labels
About Development This issue concerns the development of GAMA 👍 Fix to be tested 🧑‍🔧 Github Actions This issue concerns one repository's automated workflow

Comments

@AlexisDrogoul
Copy link
Member

Describe the bug
When submitting a pull request, a number of verifications on the code quality are made. While there can be a discussion on the interest of this (as nobody seems to really care 😉), a direct consequence is the addition of long (and frankly unreadable) comments in the PR's thread for every commit, making it virtually impossible to have a discussion in this thread.

Expected behavior
Be able to run discussions in PRs, post answers to commit comments, etc. without being polluted by uninteresting automated / spam messages from codescene or other code analyzers.

Examples
In #404, we have 11 pages of codescene nonsense, which completely dilute the 2 (!) comments made by the contributor -- and probably discourage anyone to read through the thread.
In #374, I had to manually hide (marking them as "abuse") the codescene comments just to understand what the comments made by @lesquoyb were referring to.
etc.

** Proposed solution**
If it is possible, as there is a "Checks" tab for every PR and a summary of the checks at the end of every PR, I propose to simply remove the comments from codescene and other code "quality checks" from the thread of public requests.

@AlexisDrogoul AlexisDrogoul added About Development This issue concerns the development of GAMA 🧑‍🔧 Github Actions This issue concerns one repository's automated workflow labels Feb 8, 2025
@lesquoyb
Copy link
Contributor

I've updated the codescene settings to this:
Image
Now we just need to see if it's any better

@AlexisDrogoul
Copy link
Member Author

Hi -- seems to be better now ! Thank you 🙏

@lesquoyb lesquoyb moved this from Todo to To Test in GAMA 2024-11 Feb 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
About Development This issue concerns the development of GAMA 👍 Fix to be tested 🧑‍🔧 Github Actions This issue concerns one repository's automated workflow
Projects
Status: To Test
Development

No branches or pull requests

3 participants