Use null-aware elements in dev/devicelab/lib/integration_tests.dart#187852
Conversation
There was a problem hiding this comment.
Code Review
This pull request modifies the env map in dev/devicelab/lib/tasks/integration_tests.dart by using the null-aware spread operator for environment and changing the assignment of FLUTTER_ADB_PATH. No review comments were provided, and there is no feedback to report.
|
I added |
|
Please don't merge with head again, let the tests run |
|
Thank you for your contribution! Because of the volume of PRs we receive, we require that new contributors use our checklist to guide them through critical steps in creating a Flutter PR. This PR's description is missing that checklist, so it is being marked as a Draft. Please edit the PR description to add the checklist, then ensure that you have completed all of the steps. Once you've done that, please mark the PR as ready for review. If you need help, consider asking for advice on the #hackers-new channel on Discord. |
There was a problem hiding this comment.
Code Review
This pull request updates the initialization of the env map in dev/devicelab/lib/tasks/integration_tests.dart by using the null-aware spread operator ...? for the environment variable and modifying the assignment of 'FLUTTER_ADB_PATH'. There are no review comments, so I have no feedback to provide.
Description
Adopts null-aware element syntax (
?x) indev/devicelab/lib/integration_tests.dart, replacing 2 instances of the verboseif (x != null) x!pattern in theintegration_testsfile.Part of #172188
Pre-launch Checklist
///).