-
Notifications
You must be signed in to change notification settings - Fork 272
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
v12 release #1314
v12 release #1314
Conversation
Do we have a release plan already? Starting with some RC or going straight to the stable release channel? |
I would like to RC first, as we have some breaking changes. Going this way might help use catch some potential errors before reaching general public. IIRC @AugustinLF offered to test RC on his project. |
I think a RC is a good idea, I'll also try the latest version on my project |
I've just releases @AugustinLF @pierrezimmermannbam @MattAgn could you take it for a spin, and see if it works fine with our test suits? |
I just tried it on my current project, it doesn't work with |
@pierrezimmermannbam Jest Native 5.4.2 is out with the fix! |
@pierrezimmermannbam how about other issues beside |
@mdjastrzebski I'll test with newer version of jest-native because it may have hidden other issues but so far I haven't seen other issues |
No other issues on my project, I've tested it on an older project as well, just got some byRole queries not working because they targeted non accessible elements 🎉 |
Back from vacation, I'll try to update, will keep you updated in the upcoming days :) |
I tested the RC, and apart from the type issue I mentioned in #1351 and the container problem of #1350, nothing unexpected (it did caught a few accessibility errors, and improved some of our tests that were messy because of |
Just released RC 2 🎉 |
15b228f
to
51302ce
Compare
@AugustinLF, @pierrezimmermannbam, @MattAgn the RC 2 seems to be ready from my perspective, all reported issues (#1338, ) have been fixed. Do you have any more tests to perform? Are you aware of any more issues? |
* chore: enable all breaking changes * fix: ci issues * chore: fix lint * chore: cleanup * chore: add migration guide * chore: update docs * chore: docs tweaks * chore: docs tweaks * chore: remove useBreakingChanges config flag * refactor: code review changes * docs: tweaks
Co-authored-by: Michał Pierzchała <[email protected]>
* docs: update migration guide to mention updating version of jest native * docs: add note at top level to indicate it requires an update of jest native * Update website/docs/MigrationV12.md --------- Co-authored-by: pierrezimmermann <[email protected]>
* fix: make container non-enumerable * chore: fix lint, typecheck * chore: code comment
Co-authored-by: Michał Pierzchała <[email protected]>
Co-authored-by: Michał Pierzchała <[email protected]>
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #1314 +/- ##
=======================================
Coverage 96.12% 96.13%
=======================================
Files 49 49
Lines 3359 3314 -45
Branches 503 491 -12
=======================================
- Hits 3229 3186 -43
+ Misses 130 128 -2
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
If there are no more issues and/or comments regarding this RC, let's plan release for next Tuesday, Mar 14 😄 |
Summary
This is feature branch for merging particular v12 features/toggles/changes. It will be merged to
main
branch when v12 is ready to release.useBreakingChanges
flag [v12] refactor: enable all breaking changes #1313includeHiddenElements: false
by default [v12] refactor: exclude hidden elements by defaults #1317Issues & fixes:
react-native
package #1355Test plan
All checks should pass.