Skip to content

Conversation

@lprimak
Copy link
Contributor

@lprimak lprimak commented Dec 21, 2024

Maven release 4.0.0-rc-2 changed how consumer POM is generated.
new option maven.consumer.pom=true is necessary to make ShrinkWrap work prior to the PR,
and take full advantage of maven 4 features in the future.

@lprimak lprimak marked this pull request as draft December 21, 2024 21:18
@lprimak lprimak force-pushed the fix-build-pom-access branch from e98f1d1 to f2f9803 Compare December 22, 2024 00:04
@lprimak lprimak marked this pull request as ready for review December 22, 2024 00:08
@lprimak lprimak force-pushed the fix-build-pom-access branch from f2f9803 to 1d8e66a Compare December 22, 2024 03:39
@lprimak lprimak force-pushed the fix-build-pom-access branch from 1d8e66a to 121e033 Compare December 22, 2024 04:56
@lprimak lprimak changed the title Enhancement: keep updated with maven 4 rc-2 consumer pom changes chore: keep updated with maven 4 rc-2 consumer pom changes Dec 25, 2024
@lprimak
Copy link
Contributor Author

lprimak commented Jan 10, 2025

@petrberan This can be merged first, when you come back from vacation, thank yoU!

@lprimak
Copy link
Contributor Author

lprimak commented Feb 17, 2025

@petrberan Can we start with merging this one? Thank you!

@petrberan
Copy link
Member

Hi @lprimak , terribly sorry for the delay! Truth is that after my arrival from PTO there were blocking issues with some critical infrastructure and then some work reorganization and each week I've tried to tackle Shrinkwrap issues, without much success. But I'll make sure to prioritize it next week, at least these PRs and new branch so you're not held back by my absence. Thank you for your patience though!

@petrberan petrberan merged commit e4689f5 into shrinkwrap:master Feb 26, 2025
3 checks passed
@lprimak lprimak deleted the fix-build-pom-access branch February 26, 2025 15:00
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