Skip to content

Conversation

@chingor13
Copy link
Contributor

No description provided.

@chingor13 chingor13 requested a review from jeanbza September 11, 2019 15:59
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Sep 11, 2019
@codecov
Copy link

codecov bot commented Sep 11, 2019

Codecov Report

❗ No coverage uploaded for pull request base (master@6e8a31c). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #6251   +/-   ##
=========================================
  Coverage          ?   47.19%           
  Complexity        ?    27536           
=========================================
  Files             ?     2548           
  Lines             ?   278673           
  Branches          ?    31989           
=========================================
  Hits              ?   131510           
  Misses            ?   137382           
  Partials          ?     9781

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 6e8a31c...e0c252b. Read the comment docs.

Copy link

@jeanbza jeanbza left a comment

Choose a reason for hiding this comment

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

Does this actually get used? My understanding was that some maven magic made the locally-installed jars get used.

@kolea2
Copy link
Contributor

kolea2 commented Sep 11, 2019

@jadekler yep! This version number will now get bumped every time we do a release or snapshot bump with this annotation (part of releasetool)

@jeanbza
Copy link

jeanbza commented Sep 11, 2019

Sounds good thanks!

@chingor13 chingor13 merged commit 1d211f1 into googleapis:master Sep 11, 2019
@chingor13 chingor13 deleted the storage-wrapper-dependency branch September 11, 2019 17:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants