Skip to content

util/maven: Add InterpolateRepositories for Maven Project - #311

Merged
cuixq merged 3 commits into
google:mainfrom
cuixq:maven
Nov 4, 2025
Merged

cuixq merged 3 commits into
google:mainfrom
cuixq:maven

Conversation

@cuixq

@cuixq cuixq commented Nov 3, 2025

Copy link
Copy Markdown
Collaborator

This change adds func InterpolateRepositories to Maven project so that we can still resolve the properties in Maven repositories without interpolating everything.

@michaelkedar michaelkedar left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

calling InterpolateRepositories then Interpolate doesn't cause any problems, right?

@cuixq

cuixq commented Nov 3, 2025

Copy link
Copy Markdown
Collaborator Author

I intentionally not remove any repository if it cannot be fully resolved so for later Interpolate calls these properties can still be caught up.

Comment thread util/maven/project.go Resolved
Comment thread util/maven/project.go Resolved
@cuixq
cuixq merged commit 20ad947 into google:main Nov 4, 2025
2 checks passed
@cuixq
cuixq deleted the maven branch November 4, 2025 02: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.

3 participants