-
Notifications
You must be signed in to change notification settings - Fork 6.6k
Re-enable AutoML model create tests #3721
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
Conversation
Few things.
This easily conflicts among multiple builds. Can you update this to use uuid?
|
2424525
to
202d1cb
Compare
@sirtorry I did a quick search for the region tag, and there are still used in multiple location. Although they're all translated page, can we really delete them now? |
I'm going to submit a request for these samples to be replaced on our localised/non-english documentation. We shouldn't remove the samples until that's been done. In this case, should we re-enable these tests? |
@busunkim96 Yes, it's alright! |
Closes #3706.
I don't have the context around why these tests were disabled. Does these tests need to be modified in some other way?
#1608 disabled these tests.