-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
dependency: bump unique-selector #28571
Conversation
It's a little unusual for a version Perhaps |
@MikeMcC399 Yah it's weird because I implemented semantic-versioning for the first time and it did something strange with it and I don't really want to spend more effort on it. https://github.com/cypress-io/unique-selector/releases/tag/v0.0.5 |
I guess it will work, although it is confusing especially as the https://github.com/cypress-io/unique-selector/blob/master/README.md is also not aligned to the release versions, content and contacts for the fork. |
@jennifer-shehane We need a changelog entry for this dependency bump |
4 flaky tests on run #53095 ↗︎Details:
|
Test | Artifacts | |
---|---|---|
... > dismisses the banner for a specified time |
Test Replay
Screenshots
|
specs_list_latest_runs.cy.ts • 1 flaky test • app-e2e
Test | Artifacts | |
---|---|---|
App/Cloud Integration - Latest runs and Average duration > when no runs are recorded > shows placeholders for all visible specs |
Test Replay
Screenshots
|
Review all test suite changes for PR #28571 ↗︎
Released in This comment thread has been locked. If you are still experiencing this issue after upgrading to |
Additional details
This includes a performance optimization to cypress/unique-selector, perhaps the Selector playground will be slightly faster in some cases. https://github.com/cypress-io/unique-selector/releases/tag/v0.0.5
Steps to test
Current tests should pass
How has the user experience changed?
PR Tasks
cypress-documentation
?type definitions
?