-
Notifications
You must be signed in to change notification settings - Fork 190
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
tycho-versions-plugin: Support ci-friendly versions #3900
Conversation
tycho-versions-plugin/src/main/java/org/eclipse/tycho/versions/pom/PomFile.java
Outdated
Show resolved
Hide resolved
tycho-versions-plugin/src/main/java/org/eclipse/tycho/versions/pom/PomFile.java
Outdated
Show resolved
Hide resolved
tycho-versions-plugin/src/main/java/org/eclipse/tycho/versions/pom/PomFile.java
Outdated
Show resolved
Hide resolved
31fec6c
to
19c4722
Compare
tycho-versions-plugin/src/main/java/org/eclipse/tycho/versions/pom/PomFile.java
Outdated
Show resolved
Hide resolved
In PomFile#getVersion(), resolve properties used in the raw version of the project. This is necessary so that downstream manipulators (MANIFEST.MF, feature.xml, category.xml, ...) can see the actual version and act on it. Fixes eclipse-tycho#3744.
19c4722
to
1746be2
Compare
I refactored things a bit more. In fact, the utility is not specific to versions, but just deals with properties in general. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks very good, thank you!
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation and see the Github Action logs for details |
In PomFile#getVersion(), resolve properties used in the raw version of the project. This is necessary so that downstream manipulators (MANIFEST.MF, feature.xml, category.xml, ...) can see the actual version and act on it.
Fixes #3744.