Skip to content
This repository was archived by the owner on Sep 20, 2023. It is now read-only.

chore: delete owlbot.py#230

Merged
parthea merged 4 commits intomainfrom
delete-owlbot-py
Nov 8, 2021
Merged

chore: delete owlbot.py#230
parthea merged 4 commits intomainfrom
delete-owlbot-py

Conversation

@parthea
Copy link
Copy Markdown
Contributor

@parthea parthea commented Oct 15, 2021

Now that googleapis/synthtool#1244 is merged, owlbot.py is no longer required in this repo. We can add owlbot.py back in the future if repository specific customizations are needed.

@parthea parthea requested a review from a team October 15, 2021 13:51
@product-auto-label product-auto-label bot added the api: documentai Issues related to the googleapis/python-documentai API. label Oct 15, 2021
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Oct 15, 2021
@busunkim96
Copy link
Copy Markdown
Contributor

busunkim96 commented Oct 18, 2021

Quota is exhausted, this repo may need to opt in to multiple projects for samples:

        except Exception as e:
>           assert "Failed to process" in e.message
E           AssertionError: assert 'Failed to process' in 'Resource has been exhausted (e.g. check quota).'
E            +  where 'Resource has been exhausted (e.g. check quota).' = ResourceExhausted('Resource has been exhausted (e.g. check quota).').message

batch_process_documents_sample_bad_input_test.py:44: AssertionError

@parthea
Copy link
Copy Markdown
Contributor Author

parthea commented Oct 18, 2021

I've created #231 to add multiple projects for samples.

@parthea parthea added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Oct 19, 2021
@parthea
Copy link
Copy Markdown
Contributor Author

parthea commented Oct 19, 2021

Wait for #231

@parthea parthea self-assigned this Oct 28, 2021
@parthea parthea removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Nov 6, 2021
@parthea parthea requested a review from busunkim96 November 8, 2021 12:02
@parthea parthea assigned busunkim96 and unassigned parthea Nov 8, 2021
@parthea
Copy link
Copy Markdown
Contributor Author

parthea commented Nov 8, 2021

This is ready for review.

@parthea parthea merged commit 4425463 into main Nov 8, 2021
@parthea parthea deleted the delete-owlbot-py branch November 8, 2021 16:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@busunkim96 busunkim96 busunkim96 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@busunkim96 busunkim96

Labels

api: documentai Issues related to the googleapis/python-documentai API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@parthea @busunkim96