Skip to content

Conversation

@munificent
Copy link
Member

Fix #1642.

This is using a workaround for dart-lang/sdk#60840.
That issue is fixed and a new version of analyzer is published, but
unfortunately there is no corresponding version of test that works with
it so I can't upgrade yet.

The workaround is pretty harmless, so I'm fine with it. (The formatter
uses similar logic to handle commas which also aren't represented in the
AST nodes.)
# Please enter a commit message to explain why this merge is necessary,
# especially if it merges an updated upstream into a topic branch.
#
# Lines starting with '#' will be ignored, and an empty message aborts
# the commit.
@munificent munificent requested a review from kallentu July 22, 2025 17:41
Copy link
Member

@kallentu kallentu left a comment

Choose a reason for hiding this comment

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

Nice, thanks!

Base automatically changed from dot-shorthands to main July 22, 2025 20:30
@munificent munificent merged commit 5d2ad56 into main Jul 22, 2025
1 check passed
@munificent munificent deleted the null-aware-dot-shorthand branch July 22, 2025 21:49
munificent added a commit that referenced this pull request Jul 28, 2025
…d. (#1748)

Preserve space between "?" and "." in null aware element dot shorthand.

Fix #1642.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Make sure to preserve a space between ? and .

3 participants