Skip to content
This repository was archived by the owner on Sep 26, 2023. It is now read-only.

ci: linkage-monitor-artifacts.txt generation#1320

Merged
suztomo merged 2 commits intogoogleapis:masterfrom
suztomo:artifact_list
Mar 1, 2021
Merged

ci: linkage-monitor-artifacts.txt generation#1320
suztomo merged 2 commits intogoogleapis:masterfrom
suztomo:artifact_list

Conversation

@suztomo
Copy link
Copy Markdown
Member

@suztomo suztomo commented Mar 1, 2021

Linkage Monitor by default searches for pom.xml files to get list of the artifacts that are generated by a repository. However certain repositories (such as gax-java) use Gradle and have no pom.xml files. Therefore Linkage Monitor has a functionality to let the repositories tell the list of artifacts. It reads linkage-monitor-artifacts.txt in the root of a repository.

@suztomo suztomo requested review from a team March 1, 2021 15:38
@google-cla google-cla Bot added the cla: yes This human has signed the Contributor License Agreement. label Mar 1, 2021
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 1, 2021

Codecov Report

Merging #1320 (f28808b) into master (533152a) will decrease coverage by 0.03%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1320      +/-   ##
============================================
- Coverage     79.69%   79.65%   -0.04%     
+ Complexity     1249     1248       -1     
============================================
  Files           209      209              
  Lines          5461     5461              
  Branches        464      464              
============================================
- Hits           4352     4350       -2     
- Misses          927      928       +1     
- Partials        182      183       +1     
Impacted Files Coverage Δ Complexity Δ
.../java/com/google/api/gax/batching/BatcherImpl.java 94.94% <0.00%> (-1.13%) 22.00% <0.00%> (-1.00%)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 533152a...c9dfb25. Read the comment docs.

Copy link
Copy Markdown
Contributor

@elharo elharo left a comment

Choose a reason for hiding this comment

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

I wonder if there's some way we could deduce this from the gradle files?

@suztomo
Copy link
Copy Markdown
Member Author

suztomo commented Mar 1, 2021

@elharo I wish there is. I didn't think of a better way to read values in build.gradle than coding within it in Groovy.

@suztomo suztomo merged commit 73da2af into googleapis:master Mar 1, 2021
@suztomo
Copy link
Copy Markdown
Member Author

suztomo commented Mar 1, 2021

@suztomo suztomo deleted the artifact_list branch March 1, 2021 18:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants