Skip to content
This repository has been archived by the owner on Feb 23, 2024. It is now read-only.

fix: Issue/254 #266

Closed
wants to merge 3 commits into from
Closed

fix: Issue/254 #266

wants to merge 3 commits into from

Conversation

nicain
Copy link
Contributor

@nicain nicain commented Oct 5, 2021

Fixes #254

Also moves from "flaky" to "pytest-rerunfailures", which allows for much better introspection of the aspects of flakiness.

@nicain nicain requested a review from aribray October 5, 2021 21:01
@product-auto-label product-auto-label bot added the api: translate Issues related to the googleapis/python-translate API. label Oct 5, 2021
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Oct 5, 2021
@nicain nicain marked this pull request as ready for review October 5, 2021 21:57
@nicain nicain requested review from telpirion and a team as code owners October 5, 2021 21:57
@nicain nicain requested a review from leahecole October 5, 2021 21:57
@nicain nicain added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Oct 5, 2021
@nicain
Copy link
Contributor Author

nicain commented Oct 5, 2021

@aribray Just synched up with @leahecole; we are going to try using backoff instead. I am going to close this PR for now, and work up a new one with backoff. Ill tag you as a reviewer when it is ready!

@nicain nicain closed this Oct 5, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
api: translate Issues related to the googleapis/python-translate API. cla: yes This human has signed the Contributor License Agreement. do not merge Indicates a pull request not ready for merge, due to either quality or timing.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

translate_v3_batch_translate_text_with_glossary_test: test_batch_translate_text_with_glossary failed
1 participant