Skip to content

Conversation

@goderbauer
Copy link
Member

@goderbauer goderbauer commented Jan 27, 2021

flutter create --sample=foo created a project that wasn't enabled for null-safety even though all of our samples have been migrated to null-safety. This caused analyzer warnings regarding the NNBD syntax used in the samples.

This generates a null-safe project when --sample is used.

Fixes #74564.

@flutter-dashboard flutter-dashboard bot added the tool Affects the "flutter" command-line tool. See also t: labels. label Jan 27, 2021
@google-cla google-cla bot added the cla: yes label Jan 27, 2021
Copy link
Contributor

@jonahwilliams jonahwilliams left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@goderbauer
Copy link
Member Author

@jonahwilliams @gspencergoog I am proposing in #74564 to cherry-pick this into the 1.26 release (if it doesn't make the cut) because it is breaking the flutter create --sample flow in an unintuitive way. Samples should never have analyzer warnings. Let me know if you think differently.

@gspencergoog gspencergoog changed the title Make flutter create --smaple null-safe Make flutter create --sample null-safe Jan 27, 2021
@gspencergoog gspencergoog changed the title Make flutter create --sample null-safe Make flutter create --sample null-safe Jan 27, 2021
@fluttergithubbot fluttergithubbot merged commit 63267e9 into flutter:master Jan 27, 2021
@goderbauer goderbauer deleted the createNNBD branch January 27, 2021 23:10
christopherfujino pushed a commit to chris-forks/flutter that referenced this pull request Feb 3, 2021
christopherfujino added a commit that referenced this pull request Feb 3, 2021
…#75303)

* Only show devtools deep links for render overflow errors (#74916)

* Add fixes for nullOk changes (#74866)

* [flutter_tools] delete old directories when unzipping ontop of them (#74818)

Fixes #74772

stale files from previous SDKs were getting left in the cache, confusing the analyzer.

* Make flutter create --smaple null-safe (#74844)

* Suppress 'Info: Compiling without sound null safety' message when building flutter_tools (#74740)

The CFE now logs to stdout by default when compiling a program which has
non-null-safe dependencies. Since flutter_tools has not yet migrated, we
need to suppress this message when compiling the tool.

Fixes #74366

* Split tools_tests subshards into separate shards to support sub-sub-sharding (#75033)

* Adopt tool_test shard builders (#75171)

* Adopt subshard naming convention in build_tests (#75179)

* Mark tool_integration_tests not flaky (#75273)

* roll flutter/plugins to ToT to fix analysis error

* update engine hash

Co-authored-by: Kenzie Schmoll <[email protected]>
Co-authored-by: Kate Lovett <[email protected]>
Co-authored-by: Jonah Williams <[email protected]>
Co-authored-by: Michael Goderbauer <[email protected]>
Co-authored-by: Ben Konyi <[email protected]>
Co-authored-by: Jenn Magder <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tool Affects the "flutter" command-line tool. See also t: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

flutter create --sample has null-safety warnings

4 participants