-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
feat: 14595-update-ally-test-tags #14913
feat: 14595-update-ally-test-tags #14913
Conversation
✅ Deploy Preview for v11-carbon-react ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for carbon-elements ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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! 🚀
@andreancardona seems like a few tests are timing out |
@andreancardona seems like there is a merge conflict now as well |
…595-update-ally-test-tags
Resolved 👍 |
This may need the sharding update in #14915 merged first if the tests continue to timeout here. |
@andreancardona there's a couple conflicts here - once resolved, I think this will be good to go. I solved the progress indicator errors in #14915 |
1488f59
Co-authored-by: TJ Egan <[email protected]> Co-authored-by: Guilherme Datilio Ribeiro <[email protected]> Co-authored-by: Taylor Jones <[email protected]> Co-authored-by: Taylor Jones <[email protected]>
Closes #14595
Update
avt
test naming conventions to use the following: