Skip to content
This repository has been archived by the owner on Dec 8, 2021. It is now read-only.

refactor: copy PaginationRange from spanner #168

Merged
merged 2 commits into from
Feb 10, 2020
Merged

refactor: copy PaginationRange from spanner #168

merged 2 commits into from
Feb 10, 2020

Conversation

coryan
Copy link
Contributor

@coryan coryan commented Feb 10, 2020

This change is Reviewable

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Feb 10, 2020
@codecov
Copy link

codecov bot commented Feb 10, 2020

Codecov Report

Merging #168 into master will increase coverage by 0.01%.
The diff coverage is 96.59%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #168      +/-   ##
==========================================
+ Coverage    96.2%   96.22%   +0.01%     
==========================================
  Files          92       94       +2     
  Lines        4248     4395     +147     
==========================================
+ Hits         4087     4229     +142     
- Misses        161      166       +5
Impacted Files Coverage Δ
google/cloud/internal/pagination_range.h 100% <100%> (ø)
google/cloud/internal/pagination_range_test.cc 95% <95%> (ø)

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 f4427ce...eb9bba3. Read the comment docs.

@coryan coryan marked this pull request as ready for review February 10, 2020 17:48
Copy link
Contributor

@devjgm devjgm left a comment

Choose a reason for hiding this comment

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

Should there be a .bzl file with these new files listed?

Copy link
Contributor Author

@coryan coryan left a comment

Choose a reason for hiding this comment

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

There are two .bzl files changed, but no new .bzl files:

https://github.com/googleapis/google-cloud-cpp-common/pull/168/files#diff-47d20683435f05fe192135306632a3a4

Reviewable status: 0 of 5 files reviewed, all discussions resolved (waiting on @devbww, @mr-salty, and @scotthart)

Copy link
Contributor

@devjgm devjgm left a comment

Choose a reason for hiding this comment

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

Sorry. My find-in-page-fu failed me.

@coryan coryan merged commit 2a96070 into googleapis:master Feb 10, 2020
@coryan coryan deleted the refactor-pagination-range branch February 10, 2020 18:53
coryan added a commit to coryan/google-cloud-cpp that referenced this pull request Apr 24, 2020
coryan added a commit to coryan/google-cloud-cpp that referenced this pull request Apr 24, 2020
coryan added a commit to coryan/google-cloud-cpp that referenced this pull request Apr 24, 2020
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.

3 participants