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

Improving error message in case the Cloud Billing API is not enabled #1075

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

mwatney-mars
Copy link

Changed the error message from:

print("Permission Denied, check project level permission.")

to:

print("Permission Denied, check project level permission or if the Cloud Billing API is enabled")

as the pervious one is not indicating the issue may be related to the Cloud Billing API to be disable.

@pull-request-size pull-request-size bot added the size/XS Denotes a PR that changes 0-9 lines. label Aug 21, 2023
Co-authored-by: Shakeeb Shams <shakeebshams@gmail.com>
@agold-rh
Copy link
Contributor

agold-rh commented Oct 9, 2023

@mwatney-mars Python 3 formatting check failed. Look in helpers/check_format.sh for the standard Python 3 format checking options.

@yashwantm
Copy link

yapf formatting limit the print string length can not exceed 63 char. reduce the length to pass the checks.

@agold-rh
Copy link
Contributor

/gcbrun

@agold-rh
Copy link
Contributor

@mwatney-mars Formatting checks are failing in examples/billboard/billboard.py. See the earlier comments for suggestions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/XS Denotes a PR that changes 0-9 lines.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants