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: 🐛 Fix issues #228 #233 #235 #240 #6

Merged
merged 10 commits into from
Feb 2, 2024

Conversation

alograg
Copy link
Contributor

@alograg alograg commented Jan 17, 2024

Remove depreciated rules seen in issues, refer to discussion (green-code-initiative/creedengo-rules-specifications#271)

@dedece35
Copy link
Member

dedece35 commented Jan 17, 2024

Hi @alograg,
thank you a lot for PR.
there are some lacks to remove deprecated rules : I want to write a DefinitionOfDone list (like https://github.com/green-code-initiative/ecoCode-common/blob/main/doc/starter-pack.md#definition-of-done-of-a-pr ) to check all work to do in order to delete deprecated rules (issue created for that : green-code-initiative/creedengo-common#28 )

here are lacks that I detected :

docker-compose.yml Outdated Show resolved Hide resolved
pom.xml Outdated Show resolved Hide resolved
@dedece35 dedece35 added 🔰 good first issue Good for newcomers 🔥 in progress 🔥 🗃️ rule rule improvment or rule development or bug 🚀 enhancement New feature or request labels Jan 17, 2024
Copy link
Member

@dedece35 dedece35 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please update CHANGELOG.md in "unreleased" part with these deletions.

CHANGELOG.md Outdated Show resolved Hide resolved
@alograg alograg marked this pull request as ready for review January 19, 2024 11:59
CHANGELOG.md Outdated Show resolved Hide resolved
pom.xml Outdated Show resolved Hide resolved
Remove snapshot from version od EcoCode
@dedece35
Copy link
Member

dedece35 commented Feb 1, 2024

Hi @alograg,
please take into account all my previous unresolved notes.

Once, you will correct my only one review feedback on "ecoCode-rules-specifications" repository, I will create a new version 1.5.0 of this one.
Next, this PR will be ok on Checks Tab.
(please, correct conflicts in this PR)

Thank you again !

@alograg
Copy link
Contributor Author

alograg commented Feb 2, 2024

Hi @dedece35
This is ok ?
Maybe I was wrong about something regarding the POM versions.

Copy link

sonarqubecloud bot commented Feb 2, 2024

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@dedece35 dedece35 merged commit 6c9e7eb into green-code-initiative:main Feb 2, 2024
2 checks passed
@dedece35
Copy link
Member

dedece35 commented Feb 2, 2024

Hi @alograg,
thank you again for your work.
the new 1.6.0 java release contains your work : check https://github.com/green-code-initiative/ecoCode-java/releases/tag/1.6.0

@alograg
Copy link
Contributor Author

alograg commented Feb 5, 2024

Thankyou @dedece35

I am truly humbled and honored to have received such positive feedback from you.

It has been a pleasure to participate in the EcoCode community and I am grateful for the opportunity to contribute to such a valuable project.

I am committed to continuing to contribute to the community in any way that I can. I am always looking for new ways to improve my skills and knowledge, and I am eager to share what I have learned with others.

Thank you again for your support and encouragement. I am excited to continue working with you and the rest of the community.

jycr pushed a commit to jycr/ecoCode-java that referenced this pull request Aug 29, 2024
…va repo (deletion of deprecated java rules EC4, EC53, EC63 and EC75)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔰 good first issue Good for newcomers 🗃️ rule rule improvment or rule development or bug 🔥 in progress 🔥 🚀 enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants