Skip to content

Add spanner adapter layer for ListBaselineStatusCounts#1142

Merged
jcscottiii merged 1 commit intomainfrom
jcscottiii/glue-layer-baseline-counts
Feb 10, 2025
Merged

Add spanner adapter layer for ListBaselineStatusCounts#1142
jcscottiii merged 1 commit intomainfrom
jcscottiii/glue-layer-baseline-counts

Conversation

@jcscottiii
Copy link
Collaborator

@jcscottiii jcscottiii commented Feb 6, 2025

Depends on #1132

This is the adapter layer for ListBaselineStatusCounts between the http and spanner layers.

This is similar to ListMissingOneImplCounts (also used on the stats page)

Copy link
Collaborator

@DanielRyanSmith DanielRyanSmith left a comment

Choose a reason for hiding this comment

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

I just noticed that GitHub displays these PRs with 8-space block indentation. I see 4-space blocks locally. I'm not sure why this is 🤔 maybe specific to Go?

Base automatically changed from jcscottiii/baseline_count to main February 10, 2025 15:47
This is the adapter layer for ListBaselineStatusCounts between the
http and spanner layers.

This is similar to TestListMissingOneImplCounts
@jcscottiii jcscottiii force-pushed the jcscottiii/glue-layer-baseline-counts branch from 397ba9c to cda7cd0 Compare February 10, 2025 15:55
@jcscottiii jcscottiii added this pull request to the merge queue Feb 10, 2025
@jcscottiii
Copy link
Collaborator Author

I just noticed that GitHub displays these PRs with 8-space block indentation. I see 4-space blocks locally. I'm not sure why this is 🤔 maybe specific to Go?

Never noticed it before. I wonder if it is specific to GitHub's rendering.

Merged via the queue into main with commit b48995c Feb 10, 2025
6 checks passed
@jcscottiii jcscottiii deleted the jcscottiii/glue-layer-baseline-counts branch February 10, 2025 16:56
@jcscottiii jcscottiii mentioned this pull request Feb 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants