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

fix: more sonar cloud issues #862

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

niermann999
Copy link
Contributor

@niermann999 niermann999 commented Oct 16, 2024

Fix more sonar cloud issues, picking up some leftovers in core, but mostly io, tests, tutorials etc.
This encompasses renaming python variables according to convention and adding const to global variables in the ROOT macros.

@niermann999 niermann999 added refactor refactoring the current codes priority: low Low priority labels Oct 16, 2024
@niermann999 niermann999 force-pushed the fix-sonarcloud-issues branch 4 times, most recently from 666719c to d674273 Compare October 22, 2024 23:23
@niermann999 niermann999 marked this pull request as ready for review October 22, 2024 23:23
@niermann999 niermann999 force-pushed the fix-sonarcloud-issues branch 3 times, most recently from 0cbef52 to 8228538 Compare October 23, 2024 12:27
Copy link

sonarcloud bot commented Oct 23, 2024

Quality Gate Passed Quality Gate passed

Issues
7 New issues
14 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
1.1% Duplication on New Code

See analysis details on SonarCloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: low Low priority refactor refactoring the current codes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant