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

safely remove libs version reference #10191

Conversation

renanfranca
Copy link
Contributor

@renanfranca renanfranca commented Jul 2, 2024

Fix #10060 (see the TO-DO at issue description)

…ion and entry in versions section when remove dependency
…ry when removed dependency is in build gradle profile file
…s section but kept in versions section if still used by another dependency
… version before trying to remove the version
…section but retain in versions section if used by dependency in build gradle profile file
…ion and entry in versions section when remove dependency management
…ry when removed dependency management is in build gradle profile file
…section but retain in versions section if used by another dependency management
…section but retain in versions section if used by dependency management in build gradle profile file
@renanfranca renanfranca force-pushed the 10060-safely-remove-libs-version-reference branch from f868452 to fdb31db Compare July 3, 2024 13:33
@renanfranca renanfranca marked this pull request as ready for review July 3, 2024 14:46
@renanfranca
Copy link
Contributor Author

@murdos : I generated a sampleapp with gradle using this pull request implementation and it removed the unused versions (mockito, assertj and junit-jupiter)

@pascalgrimaud pascalgrimaud added $$ bug-bounty $$ https://www.jhipster.tech/bug-bounties/ $300 https://www.jhipster.tech/bug-bounties/ labels Jul 3, 2024
@renanfranca renanfranca requested a review from murdos July 3, 2024 15:30
Copy link
Contributor

@murdos murdos left a comment

Choose a reason for hiding this comment

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

Awesome, thanks! :)

@renanfranca
Copy link
Contributor Author

Awesome, thanks! :)

Thanks 😊

@renanfranca renanfranca merged commit c4efc64 into jhipster:main Jul 3, 2024
34 checks passed
@renanfranca renanfranca deleted the 10060-safely-remove-libs-version-reference branch July 3, 2024 20:28
@renanfranca
Copy link
Contributor Author

renanfranca commented Jul 3, 2024

@murdos
Copy link
Contributor

murdos commented Jul 4, 2024

@murdos : Is it a generic error? https://github.com/jhipster/jhipster-lite/actions/runs/9783882769/job/27014572353#step:21:1 Screenshot_20240703_180007_GitHub.jpg

@renanfranca : Sorry, I've no idea 🤷‍♂️

@renanfranca
Copy link
Contributor Author

I will let it go for now and keep watching it. I tried to rerun all jobs and got a Sonar not initialized error, but when I ran it on localhost, it worked fine.
https://github.com/jhipster/jhipster-lite/actions/runs/9783882769/job/27041337459

@renanfranca
Copy link
Contributor Author

@pascalgrimaud
Copy link
Member

@renanfranca : approved

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: enhancement 🔧 $$ bug-bounty $$ https://www.jhipster.tech/bug-bounties/ generator: internal theme: gradle $300 https://www.jhipster.tech/bug-bounties/
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Gradle: removeDependency doesn't remove unused version at libs.versions.toml
3 participants