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

Renaming to Creedengo #38

Merged
merged 9 commits into from
Jan 7, 2025
Merged

Renaming to Creedengo #38

merged 9 commits into from
Jan 7, 2025

Conversation

zippy1978
Copy link
Contributor

Work done 👍

  • Refactored package names
  • Renamed resources
  • Moved to new creedengo-rules-specification
  • Updated documentation
  • Moved to version 2.0.0-SNAPSHOT

Tested with https://github.com/green-code-initiative/creedengo-mobile-ios-swift-test-project

@zippy1978 zippy1978 requested a review from a team December 10, 2024 12:27
@zippy1978 zippy1978 mentioned this pull request Dec 10, 2024
@zippy1978
Copy link
Contributor Author

@dedece35,
I renamed the Sonarqube key to'green-code-initiative_creedengo-ios'.
Could you create it on SonarCloud ?

@dedece35
Copy link
Member

@dedece35, I renamed the Sonarqube key to'green-code-initiative_creedengo-ios'. Could you create it on SonarCloud ?

done ... rerun and it's ok on sonarcloud.io

@zippy1978 zippy1978 requested a review from dedece35 December 24, 2024 09:30
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.

Hi @zippy1978,
CHANGELOG.md update missing.
apart from that, all code is OK for me.

If you have time, SonarQube raises 14 issues : check https://sonarcloud.io/project/issues?id=green-code-initiative_creedengo-ios&pullRequest=38&issueStatuses=OPEN,CONFIRMED&sinceLeakPeriod=true

@zippy1978 zippy1978 requested a review from dedece35 January 6, 2025 16:40
@zippy1978 zippy1978 merged commit 4a9d2cf into main Jan 7, 2025
3 checks passed
@zippy1978 zippy1978 deleted the feature/renaming branch January 7, 2025 09:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants