Skip to content

[in_app_purchase] Make the _FeatureCard constructor const in the Android example app - #4162

Merged
auto-submit[bot] merged 1 commit into
flutter:mainfrom
jason-simmons:analyzer_workaround_060723
Jun 8, 2023
Merged

auto-submit[bot] merged 1 commit into
flutter:mainfrom
jason-simmons:analyzer_workaround_060723

Conversation

@jason-simmons

Copy link
Copy Markdown
Member

This works around a Dart analyzer error that is blocking the engine->framework roll (see flutter/flutter#128476)

…oid example app

This works around a Dart analyzer error that is blocking the engine->framework
roll (see flutter/flutter#128476)
@jason-simmons
jason-simmons requested a review from gmackall as a code owner June 8, 2023 03:46
@flutter-dashboard

Copy link
Copy Markdown

It looks like this pull request may not have tests. Please make sure to add tests before merging. If you need an exemption to this rule, contact Hixie on the #hackers channel in Chat (don't just cc him here, he won't see it! He's on Discord!).

If you are not sure if you need tests, consider this rule of thumb: the purpose of a test is to make sure someone doesn't accidentally revert the fix. Ask yourself, is there anything in your PR that you feel it is important we not accidentally revert back to how it was before your fix?

Reviewers: Read the Tree Hygiene page and make sure this patch meets those guidelines before LGTMing.

@eyebrowsoffire eyebrowsoffire left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@zanderso
zanderso requested a review from stuartmorgan-g June 8, 2023 13:36
@zanderso

zanderso commented Jun 8, 2023

Copy link
Copy Markdown
Member

@stuartmorgan for advice on the "No version bump" check.

@stuartmorgan-g

Copy link
Copy Markdown
Collaborator

Generally we want the example code on pub.dev to be analysis-clean, but since this is an implementation package, where we don't generally expect the examples to be intended for plugin clients it's fine to treat this as a dev dependency.

@stuartmorgan-g stuartmorgan-g added override: no versioning needed Override the check requiring version bumps for most changes override: no changelog needed Override the check requiring CHANGELOG updates for most changes labels Jun 8, 2023
@jason-simmons jason-simmons added the autosubmit Merge PR when tree becomes green via auto submit App label Jun 8, 2023
@auto-submit
auto-submit Bot merged commit 1057ab4 into flutter:main Jun 8, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jun 8, 2023
auto-submit Bot pushed a commit to flutter/flutter that referenced this pull request Jun 8, 2023
flutter/packages@a84b2c2...e13b8c4

2023-06-08 [email protected] [tool] Only run unit tests in Chrome for inline web (flutter/packages#4153)
2023-06-08 [email protected] [in_app_purchase] Make the _FeatureCard constructor const in the Android example app (flutter/packages#4162)
2023-06-08 [email protected] [shared_preferences] Fix initialization race (flutter/packages#4159)
2023-06-07 [email protected] [go_router] Refactors imperative APIs and browser history (flutter/packages#4134)
2023-06-07 [email protected] [various] Add `http` 1.0 compatibility (flutter/packages#4147)
2023-06-07 [email protected] [go_router_builder] Accept required parameters not in path (flutter/packages#4039)
2023-06-07 [email protected] Roll Flutter from 0b74153 to 8a5c22e (46 revisions) (flutter/packages#4160)
2023-06-07 [email protected] [pigeon] Require analyzer 5.13.0, prepare for NamedType refactoring. (flutter/packages#4127)
2023-06-07 [email protected] Roll Flutter (stable) from f92f441 to 682aa38 (1 revision) (flutter/packages#4157)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages-flutter-autoroll
Please CC [email protected],[email protected] on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
@reidbaker reidbaker mentioned this pull request Aug 4, 2023
10 of 11 tasks
creatorpiyush pushed a commit to creatorpiyush/packages that referenced this pull request Jun 10, 2026
…oid example app (flutter#4162)

This works around a Dart analyzer error that is blocking the engine->framework roll (see flutter/flutter#128476)
bisor0627 pushed a commit to bisor0627/packages that referenced this pull request Jun 19, 2026
…oid example app (flutter#4162)

This works around a Dart analyzer error that is blocking the engine->framework roll (see flutter/flutter#128476)
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 needs tests override: no changelog needed Override the check requiring CHANGELOG updates for most changes override: no versioning needed Override the check requiring version bumps for most changes p: in_app_purchase platform-android

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants