Skip to content

Use curly_braces_in_flow_control_structures for sample code#104617

Merged
guidezpl merged 4 commits intoflutter:masterfrom
guidezpl:curlyy
May 26, 2022
Merged

Use curly_braces_in_flow_control_structures for sample code#104617
guidezpl merged 4 commits intoflutter:masterfrom
guidezpl:curlyy

Conversation

@guidezpl
Copy link
Member

@guidezpl guidezpl commented May 25, 2022 •
edited
Loading

Part of #104498

Pre-launch Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I read the Tree Hygiene wiki page, which explains my responsibilities.
  • I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
  • I signed the CLA.
  • I listed at least one issue that this PR fixes in the description above.
  • I updated/added relevant documentation (doc comments with ///).
  • I added new tests to check the change I am making, or this PR is test-exempt.
  • All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@guidezpl guidezpl requested a review from goderbauer May 25, 2022 14:50
@flutter-dashboard flutter-dashboard bot added d: api docs Issues with https://api.flutter.dev/ d: examples Sample code and demos documentation f: material design flutter/packages/flutter/material repository. c: contributor-productivity Team-specific productivity, code health, technical debt. labels May 25, 2022
@guidezpl guidezpl mentioned this pull request May 25, 2022
Closed
11 tasks
guidezpl
guidezpl commented May 25, 2022
View reviewed changes
Copy link
Member Author

@guidezpl guidezpl left a comment

Choose a reason for hiding this comment

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

Diff to review is guidezpl@fffe900, the rest is from updating packages.

@guidezpl
Copy link
Member Author

guidezpl commented May 25, 2022

The addition to examples/api/pubspec.yaml gets wiped when updating packages, what to do?

@goderbauer
Copy link
Member

goderbauer commented May 25, 2022

The addition to examples/api/pubspec.yaml gets wiped when updating packages, what to do?

Likely because of this: guidezpl@fffe900#r74497341

@goderbauer
Copy link
Member

goderbauer commented May 25, 2022

Looks like the only difference is that flutter_lints is enforcing the curly_braces_in_flow_control_structures lint. I thought you are adding to the set of lints enforced in the repository as part of #104498, so that difference would go away soon.

I actually prefer the stricter set of lints we have enabled in the repository over flutter_lints because I think it produces even more readable sample code (e.g. you have to type everything). I believe, every lint that's part of flutter_lints is also already enforced in the repository (with the exception of the curly lint mentioned, but that's hopefully about to change). So, code that's clean under the repository lints should also be clean under flutter_lints (but not necessarily the other way around).

So, overall, I think switching the samples to flutter_lints would be a step back.

@guidezpl
Copy link
Member Author

guidezpl commented May 25, 2022

Makes sense, I'll repurpose this to just fix the curly_braces_in_flow_control_structures.

@guidezpl guidezpl changed the title Use flutter_lints for sample code Use curly_braces_in_flow_control_structures for sample code May 25, 2022
@guidezpl guidezpl marked this pull request as ready for review May 25, 2022 17:25
goderbauer
goderbauer approved these changes May 25, 2022
View reviewed changes
Copy link
Member

@goderbauer goderbauer left a comment

Choose a reason for hiding this comment

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

LGTM

@guidezpl guidezpl merged commit 308b543 into flutter:master May 26, 2022
@guidezpl guidezpl deleted the curlyy branch May 26, 2022 06:59
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 27, 2022
@engine-flutter-autoroll
308b543 Use curly_braces_in_flow_control_structures for sample code (
e1132d7 
flutter/flutter#104617)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 27, 2022
@engine-flutter-autoroll
308b543 Use curly_braces_in_flow_control_structures for sample code (
99ac1cc 
flutter/flutter#104617)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 27, 2022
@engine-flutter-autoroll
308b543 Use curly_braces_in_flow_control_structures for sample code (
469b951 
flutter/flutter#104617)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 27, 2022
@engine-flutter-autoroll
308b543 Use curly_braces_in_flow_control_structures for sample code (
3dd72e6 
flutter/flutter#104617)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 27, 2022
@engine-flutter-autoroll
308b543 Use curly_braces_in_flow_control_structures for sample code (
8a1c7d4 
flutter/flutter#104617)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 27, 2022
@engine-flutter-autoroll
308b543 Use curly_braces_in_flow_control_structures for sample code (
38fd297 
flutter/flutter#104617)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 28, 2022
@engine-flutter-autoroll
308b543 Use curly_braces_in_flow_control_structures for sample code (
21835c9 
flutter/flutter#104617)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 28, 2022
@engine-flutter-autoroll
308b543 Use curly_braces_in_flow_control_structures for sample code (
7861409 
flutter/flutter#104617)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 29, 2022
@engine-flutter-autoroll
308b543 Use curly_braces_in_flow_control_structures for sample code (
94bc6c7 
flutter/flutter#104617)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 29, 2022
@engine-flutter-autoroll
308b543 Use curly_braces_in_flow_control_structures for sample code (
674d575 
flutter/flutter#104617)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 29, 2022
@engine-flutter-autoroll
308b543 Use curly_braces_in_flow_control_structures for sample code (
67652b1 
flutter/flutter#104617)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 29, 2022
@engine-flutter-autoroll
308b543 Use curly_braces_in_flow_control_structures for sample code (
f1d1de7 
flutter/flutter#104617)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 29, 2022
@engine-flutter-autoroll
308b543 Use curly_braces_in_flow_control_structures for sample code (
9130245 
flutter/flutter#104617)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 29, 2022
@engine-flutter-autoroll
308b543 Use curly_braces_in_flow_control_structures for sample code (
f0145c5 
flutter/flutter#104617)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 29, 2022
@engine-flutter-autoroll
308b543 Use curly_braces_in_flow_control_structures for sample code (
aa3d6af 
flutter/flutter#104617)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 30, 2022
@engine-flutter-autoroll
308b543 Use curly_braces_in_flow_control_structures for sample code (
8b0fa17 
flutter/flutter#104617)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 30, 2022
@engine-flutter-autoroll
308b543 Use curly_braces_in_flow_control_structures for sample code (
9eefc1d 
flutter/flutter#104617)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 30, 2022
@engine-flutter-autoroll
308b543 Use curly_braces_in_flow_control_structures for sample code (
2ef5050 
flutter/flutter#104617)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 30, 2022
@engine-flutter-autoroll
308b543 Use curly_braces_in_flow_control_structures for sample code (
c050e8a 
flutter/flutter#104617)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 30, 2022
@engine-flutter-autoroll
308b543 Use curly_braces_in_flow_control_structures for sample code (
4c0a9e7 
flutter/flutter#104617)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 30, 2022
@engine-flutter-autoroll
308b543 Use curly_braces_in_flow_control_structures for sample code (
6c10b4f 
flutter/flutter#104617)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 30, 2022
@engine-flutter-autoroll
308b543 Use curly_braces_in_flow_control_structures for sample code (
4fe25ae 
flutter/flutter#104617)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 30, 2022
@engine-flutter-autoroll
308b543 Use curly_braces_in_flow_control_structures for sample code (
4971039 
flutter/flutter#104617)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 30, 2022
@engine-flutter-autoroll
308b543 Use curly_braces_in_flow_control_structures for sample code (
81e5fe3 
flutter/flutter#104617)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 30, 2022
@engine-flutter-autoroll
308b543 Use curly_braces_in_flow_control_structures for sample code (
fb185e1 
flutter/flutter#104617)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 30, 2022
@engine-flutter-autoroll
308b543 Use curly_braces_in_flow_control_structures for sample code (
2fb0a4a 
flutter/flutter#104617)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 30, 2022
@engine-flutter-autoroll
308b543 Use curly_braces_in_flow_control_structures for sample code (
afda817 
flutter/flutter#104617)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 30, 2022
@engine-flutter-autoroll
308b543 Use curly_braces_in_flow_control_structures for sample code (
01e8e5c 
flutter/flutter#104617)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 31, 2022
@engine-flutter-autoroll
308b543 Use curly_braces_in_flow_control_structures for sample code (
056b9dc 
flutter/flutter#104617)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 31, 2022
@engine-flutter-autoroll
308b543 Use curly_braces_in_flow_control_structures for sample code (
28cfcac 
flutter/flutter#104617)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 31, 2022
@engine-flutter-autoroll
308b543 Use curly_braces_in_flow_control_structures for sample code (
04da051 
flutter/flutter#104617)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 31, 2022
@engine-flutter-autoroll
308b543 Use curly_braces_in_flow_control_structures for sample code (
84a9985 
flutter/flutter#104617)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 31, 2022
@engine-flutter-autoroll
308b543 Use curly_braces_in_flow_control_structures for sample code (
5957574 
flutter/flutter#104617)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 31, 2022
@engine-flutter-autoroll
308b543 Use curly_braces_in_flow_control_structures for sample code (
7a054ef 
flutter/flutter#104617)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 31, 2022
@engine-flutter-autoroll
308b543 Use curly_braces_in_flow_control_structures for sample code (
f698931 
flutter/flutter#104617)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 31, 2022
@engine-flutter-autoroll
308b543 Use curly_braces_in_flow_control_structures for sample code (
a5c7337 
flutter/flutter#104617)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 31, 2022
@engine-flutter-autoroll
308b543 Use curly_braces_in_flow_control_structures for sample code (
ea83801 
flutter/flutter#104617)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 31, 2022
@engine-flutter-autoroll
308b543 Use curly_braces_in_flow_control_structures for sample code (
b0cdf99 
flutter/flutter#104617)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@goderbauer goderbauer goderbauer approved these changes

Assignees

No one assigned

Labels

c: contributor-productivity Team-specific productivity, code health, technical debt. d: api docs Issues with https://api.flutter.dev/ d: examples Sample code and demos f: material design flutter/packages/flutter/material repository.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@guidezpl @goderbauer