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

EC72 Java Example Code #361

Merged
merged 3 commits into from
Jan 26, 2025

Conversation

lbg-pete
Copy link
Contributor

The EC72 example Java compliant code doesn't compile and breaks other SonarQube rules that would prevent introducing a vulnerability to SQL Injection.

This suggested contribution, if you agree, reworks the example complaint code so that it correctly compiles, conforms to other SonarQube rules and still results in the energy efficiency saving that the rule is intended to give.

mccorrip and others added 2 commits November 8, 2024 13:56
The EC72 example compliant code doesn't compile and breaks other SonarQube rules that prevent introducing vulnerability to SQL Injection.

This example complaint code conforms to other SonarQube rules, correctly compiles and still results in the efficiency saving that the rule is intended to give.
…CoE/EC72-Java-Fix

Update EC72 Example Code
@lbg-pete
Copy link
Contributor Author

Hi @dedece35, is there anything I've missed? This is my first OS contribution, so hoping I've followed the process correctly.

@utarwyn utarwyn requested a review from dedece35 December 11, 2024 14:09
@dedece35
Copy link
Member

Hi @lbg-pete,
the content of your PR is OK for me, but, please correct the current conflict.
Indeed, we renamed "ecocode" to "creedengo" (the new name of plugins) and "ECxx" to "GCIxx"
Thank you.

@lbg-pete
Copy link
Contributor Author

@dedece35 - Conflicts resolved. Thanks

@dedece35 dedece35 merged commit 85e61a4 into green-code-initiative:main Jan 26, 2025
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants