Skip to content

Conversation

@ztplz
Copy link
Contributor

@ztplz ztplz commented Mar 23, 2018

No description provided.

Copy link
Member

@xster xster left a comment

Choose a reason for hiding this comment

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

Thanks. Looks good.

I was mistaken before, the onChanged was marked as a required parameter so you do need the onChanged: null in the end

],
),
),
)
Copy link
Member

Choose a reason for hiding this comment

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

comma here

],
),
new Column(
children: <Widget>[
Copy link
Member

Choose a reason for hiding this comment

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

Seems like the linter wants you to create const lists here

@xster xster merged commit 47a576b into flutter:master Apr 4, 2018
teriyakijack pushed a commit to teriyakijack/flutter that referenced this pull request Apr 5, 2018
* upstream/master:
  Run test coverage in dart2 again (flutter#15932)
  Roll engine to 23c0b7b (flutter#16254)
  fix date/time picker borders (flutter#16240)
  Revert "Roll engine to version 58d6ea3 (flutter#16236)" (flutter#16252)
  Roll engine to version 58d6ea3 (flutter#16236)
  Add disable Cupertino Switch example (flutter#15853)
  Roll engine to ac40891 (flutter#16232)
  re-enable verify only test failure and update deps. (flutter#16090)
  Extract checking IntelliJ packages into a helper and use for Android Studio. (flutter#16198)
  Remove DevToolsSecurity check from initial flutter doctor (flutter#16007)
  export RawKeyEvent from RawKeyboardListener (flutter#15956)
  Revert "Adds Java EE module to SDK manager calls" (flutter#16227)
  update intellij metadata in template files (flutter#16211)
  Add _PointerState#toString (flutter#16197)
  add a period after xcode build done (flutter#16209)
DaveShuckerow pushed a commit to DaveShuckerow/flutter that referenced this pull request May 14, 2018
* Add disable Cupertino Switch example

* keep trailing commas consistent

* fix Cupertino Switch example linter error
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 11, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants