Skip to content
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

[native_assets_cli] Remove dry-run #1485

Open
dcharkes opened this issue Aug 30, 2024 · 0 comments
Open

[native_assets_cli] Remove dry-run #1485

dcharkes opened this issue Aug 30, 2024 · 0 comments

Comments

@dcharkes
Copy link
Collaborator

dcharkes commented Aug 30, 2024

Dry run is removed from Flutter in master. Dart 3.7 needs to be released before we can remove it in the packages here.

Getting rid of dry run would simplify the abstractions for our users.

Getting rid of dry run requires: [...]

The requirements have been fulfilled, we should remove dry run from Flutter.

@dcharkes dcharkes added this to the Native Assets v1.0 milestone Aug 30, 2024
@dcharkes dcharkes self-assigned this Aug 30, 2024
@dcharkes dcharkes moved this to In Progress in Native Assets Aug 30, 2024
@dcharkes dcharkes removed the status in Native Assets Sep 13, 2024
@dcharkes dcharkes moved this to In Progress in Native Assets Nov 22, 2024
auto-submit bot pushed a commit that referenced this issue Nov 27, 2024
This deprecates `BuildConfig.dryRun`.

We will be able to remove it after Dart 3.7 is released and we can increase the SDK bound in `native_assets_cli` to that version.

For now, add some ignores to suppress to suppress the deprecation lints.

Bug:

* #1485
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: In Progress
Development

No branches or pull requests

1 participant