8000 Add toString() method to LifecycleRule by JesseLovelace · Pull Request #6856 · googleapis/google-cloud-java · GitHub 8000
[go: up one dir, main page]

Skip to content

Conversation

@JesseLovelace
Copy link
Contributor

Adds toString() method to LifecycleRule

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Nov 20, 2019
@codecov
Copy link
codecov bot commented Nov 21, 2019

Codecov Report

Merging #6856 into master will decrease coverage by <.01%.
The diff coverage is 0%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #6856      +/-   ##
============================================
- Coverage     46.37%   46.37%   -0.01%     
  Complexity    28028    28028              
============================================
  Files          2615     2615              
  Lines        288309   288325      +16     
  Branches      33809    33809              
============================================
  Hits         133709   133709              
- Misses       144334   144350      +16     
  Partials      10266    10266
Impacted Files Coverage Δ Complexity Δ
...main/java/com/google/cloud/storage/BucketInfo.java 80.91% <0%> (-2.27%) 84 <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 9836df6...d39e65d. Read the comment docs.

@Override
public String toString() {
return MoreObjects.toStringHelper(this)
.add("action", lifecycleAction.getActionType())
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you do a toString() for ActionType? It's missing setStorageClass information in this case.

Copy link
Contributor
@frankyn frankyn left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @JesseLovelace!

@JesseLovelace JesseLovelace merged commit be2d0b6 into master Nov 22, 2019
@JesseLovelace JesseLovelace deleted the lifecycleruletostring branch November 22, 2019 18:58
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.

3 participants

0