-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
BigQuery _EnumProperty ValueError messages are not displayed properly #3520
Conversation
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed, please reply here (e.g.
|
LGTM. Thanks! Can you let us know once you've signed the CLA (unless the bot makes an automated reply)? |
I've just signed the CLA |
CLAs look good, thanks! |
@@ -318,7 +318,7 @@ def _validate(self, value): | |||
:raises: ValueError if value is not allowed. | |||
""" | |||
if value not in self.ALLOWED: | |||
raise ValueError('Pass one of: %s' ', '.join(self.ALLOWED)) |
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.
I found this very small bug in the error message, so I fixed it!