-
Notifications
You must be signed in to change notification settings - Fork 713
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
Update dart format page #6367
Update dart format page #6367
Conversation
Visit the preview URL for this PR (updated for commit e3b33a9): |
src/content/tools/dart-format.md
Outdated
@@ -66,21 +66,13 @@ so they can trigger another action in response to the exit code. | |||
$ dart format -o none --set-exit-if-changed bin/my_app.dart | |||
``` | |||
|
|||
## Use trailing commas | |||
## Whitespace and commas |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@munificent I considered the points you made in the issue and thought this might be a good in between vs. getting rid of the sections all together. Do you think a brief summary about what the format generally changes, like I added here, is useful?
I might also added to this list that format
will format the line length too (#6223) so lmk if that makes sense too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, thanks! One fix and an optional consideration :)
Co-authored-by: Parker Lougheed <[email protected]>
This looks great! Sorry I didn't see this PR go by. |
Fixes #6222