-
Notifications
You must be signed in to change notification settings - Fork 16
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
FED-2442 Merge version 3.0.0 changes to master #160
Conversation
Null safety migration
FED-2223 query(All)ByTestId null arg improvements
FED-2285 Skip testing late nullable props in common component test utils
Security InsightsNo security relevant content was detected by automated scans. Action Items
Questions or Comments? Reach out on Slack: #support-infosec. |
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.
Just some comments on the changelog, but we can always update those as we're ready to release.
+10
* The first argument of `queryByTestId` / `queryAllByTestId` (`root`) is now non-nullable ([#157](https://github.com/Workiva/over_react_test/pull/157)). | ||
* SDK / Dependency Versioning: | ||
* Increase minimum Dart SDK version from `2.11.0` to `2.13.0`. | ||
* Increase minimum `react` version from `6.0.1` to `7.0.0`. |
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.
#nit This will also have over_react ^5.0.0
once it's released, so we could also add that now
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.
I'll make a note to add that in the actual release PR
FED-2451 Make root args to query(All)ByTestId nullable
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.
+10
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.
+1 to greg's recent commits
RM +1 |
@Workiva/release-management-pp |
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.
+1 from RM
As a first step to releasing version 3.0.0 of this library, we need to:
v3_wip
branch into master without actually tagging it as a release.Then, once
over_react
version 5 has been released, we can consume it and release version 3 of this library while bumping the minimumover_react
version at the same time.