Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support SetMaxNamePath API #117

Merged
merged 2 commits into from
Jun 18, 2023
Merged

Support SetMaxNamePath API #117

merged 2 commits into from
Jun 18, 2023

Conversation

goccy
Copy link
Owner

@goccy goccy commented Jun 18, 2023

ref goccy/bigquery-emulator#139

Support to set the maximum value of name path

  • If we have specified a valid full path as name path, so that the value can be used as is.

@codecov-commenter
Copy link

codecov-commenter commented Jun 18, 2023

Codecov Report

Merging #117 (a4cf174) into main (6a475dd) will decrease coverage by 0.09%.
The diff coverage is 59.79%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #117      +/-   ##
==========================================
- Coverage   45.65%   45.56%   -0.09%     
==========================================
  Files          46       47       +1     
  Lines       17715    17739      +24     
==========================================
- Hits         8087     8082       -5     
- Misses       8162     8189      +27     
- Partials     1466     1468       +2     

@goccy goccy merged commit 3664acb into main Jun 18, 2023
7 checks passed
@goccy goccy deleted the support-max-name-path branch June 18, 2023 10:45
ohaibbq pushed a commit to Recidiviz/go-zetasqlite that referenced this pull request Jan 23, 2024
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.

None yet

2 participants