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

Conversation

@elharo
Copy link
Contributor

@elharo elharo commented Jul 13, 2018

Tried to upgrade guava too but that ran into issues with errorprone annotations

@garrettjonesgoogle
Copy link
Member

The Java 7 builds appear to be failing for a different reason - I filed https://github.com/googleapis/api-common-java/issues/58 to get them fixed; we should probably do that before submitting this.

@garrettjonesgoogle garrettjonesgoogle self-requested a review July 19, 2018 17:11
@elharo
Copy link
Contributor Author

elharo commented Jul 19, 2018

Makes sense

@codecov-io
Copy link

codecov-io commented Jul 30, 2018

Codecov Report

Merging #57 into master will increase coverage by 0.16%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master      #57      +/-   ##
============================================
+ Coverage     61.01%   61.17%   +0.16%     
  Complexity      145      145              
============================================
  Files            14       14              
  Lines           613      613              
  Branches         91       92       +1     
============================================
+ Hits            374      375       +1     
  Misses          215      215              
+ Partials         24       23       -1
Impacted Files Coverage Δ Complexity Δ
...java/com/google/api/pathtemplate/PathTemplate.java 64.68% <0%> (+0.29%) 86% <0%> (ø) ⬇️

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 7728252...476d2c0. Read the comment docs.

Copy link
Member

@garrettjonesgoogle garrettjonesgoogle left a comment

Choose a reason for hiding this comment

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

LGTM

@garrettjonesgoogle garrettjonesgoogle merged commit 6f8ff7d into googleapis:master Jul 30, 2018
@elharo elharo deleted the deps branch July 30, 2018 17:54
miraleung pushed a commit that referenced this pull request Jun 4, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants