-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Move implicit majyk inside 'Client.__init__'. #956
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
Conversation
gcloud/datastore/client.py
Outdated
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
|
This seems pretty straightforward, thanks for making the change tiny and digestible :) LGTM other than the comments I've already made. |
Move implicit majyk inside 'Client.__init__'.
Closes #955. Co-authored-by: Bu Sun Kim <[email protected]>
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
* feat: Add field information when raising validation errors. * 🦉 Updates from OwlBot post-processor See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md * fix: Fix string formatting error --------- Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
* Make code changes in _default * Add unit tests. * Fix docstring. Co-authored-by: arithmetic1728 <[email protected]>
See #944.