Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Generate syntax for plugin registration that works both with and without null safety. #109480

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

Merged

Conversation

eyebrowsoffire
Copy link
Contributor

@eyebrowsoffire eyebrowsoffire commented Aug 12, 2022

Add a language header to specify we can use null safety in the generated plugin registrant file.

This fixes #109160

@eyebrowsoffire eyebrowsoffire requested a review from ditman August 12, 2022 23:51
@flutter-dashboard flutter-dashboard bot changed the base branch from dev to master August 12, 2022 23:51
@flutter-dashboard
Copy link

This pull request was opened against a branch other than master. Since Flutter pull requests should not normally be opened against branches other than master, I have changed the base to master. If this was intended, you may modify the base back to dev. See the Release Process for information about how other branches get updated.

Reviewers: Use caution before merging pull requests to branches other than master, unless this is an intentional hotfix/cherrypick.

@flutter-dashboard flutter-dashboard bot added engine flutter/engine repository. See also e: labels. framework flutter/packages/flutter repository. See also f: labels. tool Affects the "flutter" command-line tool. See also t: labels. labels Aug 12, 2022
@eyebrowsoffire eyebrowsoffire changed the base branch from master to dev August 12, 2022 23:52
@eyebrowsoffire eyebrowsoffire changed the base branch from dev to master August 15, 2022 19:20
@eyebrowsoffire eyebrowsoffire force-pushed the register_plugins_no_null_safety branch from e86e093 to 007f96b Compare August 15, 2022 19:20
@goderbauer goderbauer removed the framework flutter/packages/flutter repository. See also f: labels. label Aug 16, 2022
Copy link
Contributor

@yjbanov yjbanov left a comment

Choose a reason for hiding this comment

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

lgtm

@eyebrowsoffire
Copy link
Contributor Author

Blocked by #109797 currently

@eyebrowsoffire eyebrowsoffire added the autosubmit Merge PR when tree becomes green via auto submit App label Aug 19, 2022
@auto-submit auto-submit bot merged commit 5aad12c into flutter:master Aug 19, 2022
@eyebrowsoffire eyebrowsoffire deleted the register_plugins_no_null_safety branch August 19, 2022 16:56
MatthewPatience pushed a commit to MatthewPatience/flutter that referenced this pull request Aug 20, 2022
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Aug 20, 2022
engine-flutter-autoroll added a commit to engine-flutter-autoroll/plugins that referenced this pull request Aug 20, 2022
eyebrowsoffire added a commit to eyebrowsoffire/flutter that referenced this pull request Aug 29, 2022
CaseyHillers pushed a commit that referenced this pull request Aug 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
autosubmit Merge PR when tree becomes green via auto submit App engine flutter/engine repository. See also e: 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.

Building web without null safety fails in 3.3.0
3 participants