-
Notifications
You must be signed in to change notification settings - Fork 291
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
E2E: Enable E2E v2 in CI #5952
base: ci-enable-e2e-v2/windows-tweaks-1
Are you sure you want to change the base?
E2E: Enable E2E v2 in CI #5952
Conversation
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
276e91f
to
71004fb
Compare
7c9f76a
to
f935409
Compare
E2E Test - SummaryDetails Open report ↗︎ Failed testse2e › e2e/features/command-edit.test.ts › edit command › can change edit ranges |
Before E2Ev2 tests were not running as part of CI as to not introduce unnecessary warnings/errors whilst they still received some polish. Now that E2Ev2 tests have stabalized (and all remaining E2E tests have a upcoming migration PR) they are stable enough to run in CI.
Now we move all the current E2E tests to legacy (we might need to update the Required Merge Action Names) and enable E2Ev2 tests in CI.
Once all remaining tests have migrated (upcoming PR) we can make this new CI step required for merging.
Test plan