Skip to content

feat(billingbudgets): initial generation of billing budget - #9622

Merged
busunkim96 merged 4 commits into
googleapis:masterfrom
busunkim96:billing-budget-v1beta1
Nov 14, 2019
Merged

feat(billingbudgets): initial generation of billing budget#9622
busunkim96 merged 4 commits into
googleapis:masterfrom
busunkim96:billing-budget-v1beta1

Conversation

@busunkim96

@busunkim96busunkim96 commented Nov 6, 2019

Copy link
Copy Markdown
Contributor

No description provided.

@googlebotgooglebot added the cla: yes This human has signed the Contributor License Agreement. label Nov 6, 2019
@busunkim96busunkim96 changed the title feat(billingbudgets): initial generation of billing budget v1beta1feat(billingbudgets): initial generation of billing budgetNov 6, 2019
@busunkim96
busunkim96 requested a review from a teamNovember 7, 2019 00:11
tseaver
tseaver suggested changes Nov 7, 2019
View reviewed changes
Comment threadbillingbudgets/README.rst
@@ -0,0 +1,75 @@
Python Client for Cloud Billing Budget API (`Alpha`_)
=====================================================

@tseavertseaverNov 7, 2019

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needs a top-level summary description.

@busunkim96busunkim96Nov 7, 2019

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

billingbudgets.yaml is missing a summary line. I've asked the API team to add one.

@tseavertseaverNov 11, 2019

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you want to wait on that change and update the PR with the same text?

@busunkim96busunkim96Nov 11, 2019

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes I'll keep this open until I see the change to the protos

Comment threadbillingbudgets/docs/README.rst Outdated
Comment threadbillingbudgets/.repo-metadata.json Outdated
Comment threadbillingbudgets/README.rst Outdated
Comment threadbillingbudgets/README.rst Outdated
Comment threadbillingbudgets/README.rst Outdated
Comment threadbillingbudgets/setup.py Outdated
@busunkim96
busunkim96force-pushed the billing-budget-v1beta1 branch from 8faa130 to c054f77CompareNovember 7, 2019 21:22
@busunkim96
busunkim96 merged commit 6751237 into googleapis:masterNov 14, 2019
@busunkim96
busunkim96 deleted the billing-budget-v1beta1 branch November 14, 2019 20:42
This was referenced Jan 29, 2020
Merged
Closed
Closed
parthea pushed a commit that referenced this pull request Jun 4, 2023
@busunkim96
feat(billingbudgets): initial generation of billing budget (#9622)
7d985c9
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
@tseavertseavertseaver approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

cla: yesThis human has signed the Contributor License Agreement.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@busunkim96@tseaver@googlebot