-
Notifications
You must be signed in to change notification settings - Fork 6.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
fix(translate): fix a broken test #4360
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Failed Py 3.8 test with 404, but passed the others. Language version should not have mattered. Is it possible that simultaneous running tests interfered with each other? Perhaps name test based on UUID instead of timestamp. |
Yeah, I would change the test to use UUID instead of Timestamp. |
@engelke Good catch! I didn't notice it's using timestamp. I'll change it to use uuid. |
…n-docs-samples#4360) * fix(translate): fix a broken test fixes #4353 * use uuid * fix builds
…n-docs-samples#4360) * fix(translate): fix a broken test fixes #4353 * use uuid * fix builds
…n-docs-samples#4360) * fix(translate): fix a broken test fixes #4353 * use uuid * fix builds
…n-docs-samples#4360) * fix(translate): fix a broken test fixes #4353 * use uuid * fix builds
…n-docs-samples#4360) * fix(translate): fix a broken test fixes #4353 * use uuid * fix builds
…n-docs-samples#4360) * fix(translate): fix a broken test fixes #4353 * use uuid * fix builds
…n-docs-samples#4360) * fix(translate): fix a broken test fixes #4353 * use uuid * fix builds
…n-docs-samples#4360) * fix(translate): fix a broken test fixes #4353 * use uuid * fix builds
…n-docs-samples#4360) * fix(translate): fix a broken test fixes #4353 * use uuid * fix builds
fixes #4353