FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fix: strip trailing _ from field mask paths by busunkim96 · Pull Request #251 · googleapis/python-api-core · GitHub

This repository was archived by the owner on Feb 23, 2026. It is now read-only.
/ python-api-core Public archive

fix: strip trailing _ from field mask paths - #251

Merged
tseaver merged 1 commit into
v1from
add-field-mask-fix
Aug 13, 2021
Merged

fix: strip trailing _ from field mask paths#251
tseaver merged 1 commit into
v1from
add-field-mask-fix

Conversation

busunkim96 commented Aug 12, 2021
edited
Loading

Copy link
Copy Markdown
Contributor

Original changes is in #228. Since the master branch has already moved on to 2.0.0, I am planning on releasing this manually as 1.31.2 to PyPI.

Note: this is being merged into the v1 branch rather than master. Since there's a small chance we might want to push additional fixes to it in the future, I've added minimal branch protection on v1 (reviews required to merge).

busunkim96 requested a review from a team August 12, 2021 21:20
google-cla Bot added the cla: yes This human has signed the Contributor License Agreement. label Aug 12, 2021
busunkim96 changed the base branch from master to v1 August 12, 2021 21:20
busunkim96 requested a review from parthea August 12, 2021 21:22
busunkim96 changed the title fix: add field mask fix to v1 fix: strip trailing _ from field mask paths Aug 12, 2021
tseaver merged commit ccce0fe into v1 Aug 13, 2021
tseaver deleted the add-field-mask-fix branch August 13, 2021 14:59
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL