chore(deps): update dependency dart_style to ^3.1.10#3048
Open
renovate[bot] wants to merge 1 commit into
Open
Conversation
12a01d4 to
4723bad
Compare
4723bad to
f402217
Compare
f402217 to
7a369ae
Compare
7a369ae to
ca35091
Compare
ca35091 to
7d4e0a0
Compare
7d4e0a0 to
ff9aa0b
Compare
ff9aa0b to
98584b9
Compare
98584b9 to
8322b0d
Compare
9c4e72d to
e0e3c6f
Compare
10f5509 to
78b72b6
Compare
78b72b6 to
fb551f5
Compare
fb551f5 to
94a199f
Compare
94a199f to
5c715c9
Compare
5c715c9 to
4ad4deb
Compare
4ad4deb to
189959d
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR contains the following updates:
^3.1.0→^3.1.10Warning
Some dependencies could not be looked up. Check the Dependency Dashboard for more information.
Release Notes
dart-lang/dart_style (dart_style)
v3.1.10Compare Source
dart format --version --verbose.Bug fixes
analysis_options.yamlfile has anincludethat points toa non-existent or unreadable file (#1840).
Style changes
The following minor style bug fixes are not language versioned and apply to all
formatted code:
Fix a bug in eager splitting optimization that in rare cases would lead to a
collection or argument list splitting unnecessarily (#1809).
Don't add a blank line before a comment at the end of a compilation unit or
braced body (#1644).
If you have already formatted code using dart_style that encounters this bug,
then reformatting it even after this fix will have no effect since the
unneeded blank line was already added.
The following changes only apply when formatting code at language version 3.13
or higher:
Fix a bug in an eager splitting optimization that would lead the formatter to
prefer less desirable solutions (#1847).
Typically, the code affected by this bug is a call chain that contains an
argument list with a large collection literal, as in:
Prefer to split call chains for single-element targets (#1732).
When formatting a method call chain whose target can also split, the formatter
must decide whether to split the target or the call chain (or both). For
example:
We've tried various heuristics for this over the years but most make some code
look better while making other code look worse. This version introduces a
relatively simple rule that seems to work well in practice: If the call chain
target has only one element or argument, then prefer to split the call chain
and keep the target together. So in the above example, if prefers the second
output.
Allow block formatting parameter lists (#1693). The formatter supports
"block formatting" for most bracket-delimited constructs in the language. This
is what enables a multi-line list literal in an assignment to look like this:
variable = [ some, list, elements, ];Instead of:
variable = [ some, list, elements, ];This style applies to most language constructs, but function parameter lists
were omitted. Now they are not. This rarely shows up in real code, except for
typedefs of large function types:
Allow
as,is, andis!expressions to be block formatted (#1542).Separate imports into sections (#1120). Following the guidelines in
"Effective Dart", the formatter inserts a blank line between
"dart:", "package:", and other imports:
In if-case statements and elements, split the guard if the pattern
block-splits (#1596).
This tends to lead to code where the pattern is kept on one line and the
guard splits:
When no solution fits the page width, prefer solutions where the overflowing
lines have trailing string literals or comments (#1802, #1803, #1837).
Sometimes the formatter is unable to split the code in a way that fits it all
within the page width. When this happens, the formatter prefers whatever
solution has the fewest overflowing characters.
In practice, overflowing solutions are usually caused by long string literals
or comments that the user should split manually. To help the user do that, the
formatter now treats overflowing characters caused by trailing string
literals, comments, and a few other things that often follow a string literal
like
,,;,() {, or() async {, as "less bad" when comparing theamount of overflow between two solutions.
The effect is that when no solution fits, the formatter tends to prefer a
solution with hanging strings or comments, which makes it clearer to the user
which code they need to go back and manually split.
This change has no effect on code that does fit in the page width.
Write a trailing comma in split extension type representation clauses when in
a library whose language version allows it (#1845).
Prior to Dart 3.13, extension type representation clauses didn't allow
trailing commas even though they syntactically appear like formal parameter
lists. In Dart 3.13, that was fixed, so now the formatter formats them the
same way as other parameter lists in primary constructors.
Internal changes
analyzer: ^13.1.0.v3.1.9Compare Source
analyzer: ^13.0.0.v3.1.8Compare Source
Style changes
Format extension type representation clauses the same way primary constructor
formal parameter lists are formatted. This rarely makes a difference but
produces better formatting when the representation type is long and there are
other clauses on the extension type, as in:
This change is not language versioned. (The old style is always worse, and
continuing to support it would add complexity to the formatter.)
Force blank lines around a mixin or extension type declaration if it doesn't
have a
;body:The formatter already forces blank lines around class, enum, and extension
declarations. Mixins and extension types were overlooked. This makes them
consistent. This style change is language versioned and only affects
libraries at 3.13 or higher.
Note that the formatter allows classes and extension types whose body is
;to not have a blank line above or below them.
Internal changes
analyzer: '^12.0.0'.v3.1.7Compare Source
analyzer: '>=10.0.0 <12.0.0'.v3.1.6Compare Source
Style changes
;inan enum with members unless there actually is a trailing comma.
(Fix by @Barbirosha.)
Internal changes
sdk: ^3.10.0.v3.1.5Compare Source
v3.1.4Compare Source
analyzer: '^10.0.0'.v3.1.3Compare Source
No longer format imports with configurations and a prefix in the wrong order.
The parser used to accept this without error even though it violated the
language spec. The parser is being fixed, so the formatter will no longer
accept or format code like:
Don't force a space between
?and.if a null-aware element contains adot shorthand.
Require
analyzer: '>=8.2.0 <10.0.0'.Require
args: ^2.5.0.Require
sdk: ^3.9.0.Bug fixes
@dart=version comments when determining which >3.7 style to apply.The formatter correctly used those comments to switch between the old short
and new tall style, but ignored them for language versioned style rule changes
after 3.7. Now the language version of the file is consistently respected for
all style rules (#1762).
v3.1.2Compare Source
package:analyzer.Bug fixes
trailing_commas: preserve) applies to recordtype annotations too (#1721).
Style changes
This change only applies to code whose language version is 3.10 or higher:
When
trailing_commasispreserve, preserve a trailing comma after the lastenum constant when members are present (#1678, #1729).
(Thanks to jellynoone@ for this change.)
v3.1.1Compare Source
dart format --version --verbose.Bug fixes
analysis_options.yamlfile has anincludethat points toa non-existent or unreadable file (#1840).
Style changes
The following minor style bug fixes are not language versioned and apply to all
formatted code:
Fix a bug in eager splitting optimization that in rare cases would lead to a
collection or argument list splitting unnecessarily (#1809).
Don't add a blank line before a comment at the end of a compilation unit or
braced body (#1644).
If you have already formatted code using dart_style that encounters this bug,
then reformatting it even after this fix will have no effect since the
unneeded blank line was already added.
The following changes only apply when formatting code at language version 3.13
or higher:
Fix a bug in an eager splitting optimization that would led the formatter to
prefer less desirable solutions (#1847).
Typically, the code affected by this bug is a call chain that contains an
argument list with a large collection literal, as in:
Prefer to split call chains for single-element targets (#1732).
When formatting a method call chain whose target can also split, the formatter
must decide whether to split the target or the call chain (or both). For
example:
We've tried various heuristics for this over the years but most make some code
look better while making other code look worse. This version introduces a
relatively simple rule that seems to work well in practice: If the call chain
target has only one element or argument, then prefer to split the call chain
and keep the target together. So in the above example, if prefers the second
output.
Allow block formatting parameter lists (#1693). The formatter supports
"block formatting" for most bracket-delimited constructs in the language. This
is what enables a multi-line list literal in an assignment to look like this:
variable = [ some, list, elements, ];Instead of:
variable = [ some, list, elements, ];This style applies to most language constructs, but function parameter lists
were omitted. Now they are not. This rarely shows up in real code, except for
typedefs of large function types:
Allow
as,is, andis!expressions to be block formatted (#1542).Separate imports into sections (#1120). Following the guidelines in
"Effective Dart", the formatter inserts a blank line between
"dart:", "package:", and other imports:
In if-case pieces, split the guard if the pattern block-splits (#1596).
This tends to lead to code where the pattern is kept on one line and the
guard splits:
When no solution fits the page width, prefer solutions where the overflowing
lines have trailing string literals or comments (#1802, #1803, #1837).
Sometimes the formatter is unable to split the code in a way that fits it all
within the page width. When this happens, the formatter prefers whatever
solution has the fewest overflowing characters.
In practice, overflowing solutions are usually caused by long string literals
or comments that the user should split manually. To help the user do that, the
formatter now treats overflowing characters caused by trailing string
literals, comments, and a few other things that often follow a string literal
like
,,;,() {, or() async {, as "less bad" when comparing theamount of overflow between two solutions.
The effect is that when no solution fits, the formatter tends to prefer a
solution with hanging strings or comments, which makes it clearer to the user
which code they need to go back and manually split.
This change has no effect on code that does fit in the page width.
Write a trailing comma in split extension type representation clauses when in
a library whose language version allows it (#1845).
Prior to Dart 3.13, extension type representation clauses didn't allow
trailing commas even though they syntactically appear like formal parameter
lists. In Dart 3.13, that was fixed, so now the formatter formats them the
same way as other parameter lists in primary constructors.
Internal changes
analyzer: ^13.1.0.Configuration
📅 Schedule: (UTC)
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.