Skip to content

Linkage monitor to read file to get list of artifacts for a repository (when necessary)#1961

Closed
suztomo wants to merge 3 commits intoGoogleCloudPlatform:masterfrom
suztomo:linkage_monitor_read_file
Closed

Linkage monitor to read file to get list of artifacts for a repository (when necessary)#1961
suztomo wants to merge 3 commits intoGoogleCloudPlatform:masterfrom
suztomo:linkage_monitor_read_file

Conversation

@suztomo
Copy link
Copy Markdown
Contributor

@suztomo suztomo commented Feb 26, 2021

@elharo As discussed, this replaces the last month's https://github.com/GoogleCloudPlatform/cloud-opensource-java/pull/1920/files.

What is the file will look like?

gax-java repository to have linkage-monitor-artifacts.txt in the root directory.

com.google.api:gax
com.google.api:gax-grpc
com.google.api:gax-httpjson

CC: @chingor13 who owns gax-java

Why versionless coordinates?

One thing I didn't know during the discussion was that the versions of generated artifacts are not static. This PR uses RepositoryUtility.findVersions() to find the latest version for the versionless coordinates listed in linkage-monitor-artifacts.txt. (Alternatively we could modify gax-java's build.gradle to generate linkage-monitor-artifacts.txt with concrete coordinates. This seemed complex compared to RepositoryUtility.findVersions()).

Fixes #1958

@google-cla google-cla bot added the cla: yes label Feb 26, 2021
@suztomo suztomo requested a review from elharo February 26, 2021 18:05
Comment on lines +1 to +3
com.google.api:gax
com.google.api:gax-grpc
com.google.api:gax-httpjson No newline at end of file
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This is the file in gax-java repository will look like.

@suztomo
Copy link
Copy Markdown
Contributor Author

suztomo commented Feb 26, 2021

Closing this in favor of #1962

@suztomo suztomo closed this Feb 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Linkage Monitor unexpectedly recognizes the root parent pom as a BOM

1 participant