[M] Change in dart/sdk[stable]: [stable][DAS] Fixes completion for `!=` for dot shorthands

0 views
Skip to first unread message

Kallen Tu (Gerrit)

unread,
Dec 10, 2025, 2:15:19 PM12/10/25
to Leaf Petersen, Samuel Rawlins, Commit Queue, dart-analys...@google.com, rev...@dartlang.org
Attention needed from Leaf Petersen and Samuel Rawlins

Kallen Tu voted Commit-Queue+1

Commit-Queue+1
Open in Gerrit

Related details

Attention is currently required from:
  • Leaf Petersen
  • Samuel Rawlins
Submit Requirements:
  • requirement satisfiedChangelog
  • requirement satisfiedCherry-Pick
  • requirement is not satisfiedCherry-Pick-Review
  • requirement is not satisfiedCode-Review
  • requirement is not satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: sdk
Gerrit-Branch: stable
Gerrit-Change-Id: Ic3a7e0e31a2296aaef121ba6d752c6ef55c041d8
Gerrit-Change-Number: 467384
Gerrit-PatchSet: 2
Gerrit-Owner: Kallen Tu <kall...@google.com>
Gerrit-Reviewer: Kallen Tu <kall...@google.com>
Gerrit-Reviewer: Leaf Petersen <le...@google.com>
Gerrit-Reviewer: Samuel Rawlins <sraw...@google.com>
Gerrit-Attention: Leaf Petersen <le...@google.com>
Gerrit-Attention: Samuel Rawlins <sraw...@google.com>
Gerrit-Comment-Date: Wed, 10 Dec 2025 19:15:16 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
unsatisfied_requirement
open
diffy

Samuel Rawlins (Gerrit)

unread,
Dec 10, 2025, 2:15:36 PM12/10/25
to Kallen Tu, Leaf Petersen, Commit Queue, dart-analys...@google.com, rev...@dartlang.org
Attention needed from Kallen Tu and Leaf Petersen

Samuel Rawlins voted Code-Review+1

Code-Review+1
Open in Gerrit

Related details

Attention is currently required from:
  • Kallen Tu
  • Leaf Petersen
Submit Requirements:
    • requirement satisfiedChangelog
    • requirement satisfiedCherry-Pick
    • requirement is not satisfiedCherry-Pick-Review
    • requirement satisfiedCode-Review
    • requirement satisfiedReview-Enforcement
    Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
    Gerrit-MessageType: comment
    Gerrit-Project: sdk
    Gerrit-Branch: stable
    Gerrit-Change-Id: Ic3a7e0e31a2296aaef121ba6d752c6ef55c041d8
    Gerrit-Change-Number: 467384
    Gerrit-PatchSet: 2
    Gerrit-Owner: Kallen Tu <kall...@google.com>
    Gerrit-Reviewer: Kallen Tu <kall...@google.com>
    Gerrit-Reviewer: Leaf Petersen <le...@google.com>
    Gerrit-Reviewer: Samuel Rawlins <sraw...@google.com>
    Gerrit-Attention: Leaf Petersen <le...@google.com>
    Gerrit-Attention: Kallen Tu <kall...@google.com>
    Gerrit-Comment-Date: Wed, 10 Dec 2025 19:15:34 +0000
    Gerrit-HasComments: No
    Gerrit-Has-Labels: Yes
    satisfied_requirement
    unsatisfied_requirement
    open
    diffy

    Leaf Petersen (Gerrit)

    unread,
    Dec 10, 2025, 3:41:52 PM12/10/25
    to Kallen Tu, Samuel Rawlins, Commit Queue, dart-analys...@google.com, rev...@dartlang.org
    Attention needed from Kallen Tu

    Leaf Petersen voted Code-Review+1

    Code-Review+1
    Open in Gerrit

    Related details

    Attention is currently required from:
    • Kallen Tu
    Submit Requirements:
      • requirement satisfiedChangelog
      • requirement satisfiedCherry-Pick
      • requirement satisfiedCherry-Pick-Review
      • requirement satisfiedCode-Review
      • requirement satisfiedReview-Enforcement
      Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
      Gerrit-MessageType: comment
      Gerrit-Project: sdk
      Gerrit-Branch: stable
      Gerrit-Change-Id: Ic3a7e0e31a2296aaef121ba6d752c6ef55c041d8
      Gerrit-Change-Number: 467384
      Gerrit-PatchSet: 2
      Gerrit-Owner: Kallen Tu <kall...@google.com>
      Gerrit-Reviewer: Kallen Tu <kall...@google.com>
      Gerrit-Reviewer: Leaf Petersen <le...@google.com>
      Gerrit-Reviewer: Samuel Rawlins <sraw...@google.com>
      Gerrit-Attention: Kallen Tu <kall...@google.com>
      Gerrit-Comment-Date: Wed, 10 Dec 2025 20:41:49 +0000
      Gerrit-HasComments: No
      Gerrit-Has-Labels: Yes
      satisfied_requirement
      open
      diffy

      Kallen Tu (Gerrit)

      unread,
      Dec 10, 2025, 4:20:36 PM12/10/25
      to Leaf Petersen, Samuel Rawlins, Commit Queue, dart-analys...@google.com, rev...@dartlang.org

      Kallen Tu voted Commit-Queue+2

      Commit-Queue+2
      Open in Gerrit

      Related details

      Attention set is empty
      Submit Requirements:
      • requirement satisfiedChangelog
      • requirement satisfiedCherry-Pick
      • requirement satisfiedCherry-Pick-Review
      • requirement satisfiedCode-Review
      • requirement satisfiedReview-Enforcement
      Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
      Gerrit-MessageType: comment
      Gerrit-Project: sdk
      Gerrit-Branch: stable
      Gerrit-Change-Id: Ic3a7e0e31a2296aaef121ba6d752c6ef55c041d8
      Gerrit-Change-Number: 467384
      Gerrit-PatchSet: 2
      Gerrit-Owner: Kallen Tu <kall...@google.com>
      Gerrit-Reviewer: Kallen Tu <kall...@google.com>
      Gerrit-Reviewer: Leaf Petersen <le...@google.com>
      Gerrit-Reviewer: Samuel Rawlins <sraw...@google.com>
      Gerrit-Comment-Date: Wed, 10 Dec 2025 21:20:32 +0000
      Gerrit-HasComments: No
      Gerrit-Has-Labels: Yes
      satisfied_requirement
      open
      diffy

      Commit Queue (Gerrit)

      unread,
      Dec 10, 2025, 4:20:53 PM12/10/25
      to Kallen Tu, Leaf Petersen, Samuel Rawlins, dart-analys...@google.com, rev...@dartlang.org

      Commit Queue submitted the change

      Change information

      Commit message:
      [stable][DAS] Fixes completion for `!=`  for dot shorthands

      Issue description: Code completion for != wasn't using the correct
      context so no dot shorthands were suggested.

      What is the fix: Fixed a typo in the context type provider in the
      analysis server. `BANG_EQ_EQ` -> `BANG_EQ`

      Why cherry-pick: Users can now see code completions of shorthands
      with `!=`.

      Risk: Low, this is a small typo fix to the correct operator and
      would only affect code completion.

      Issue link(s): https://github.com/dart-lang/sdk/issues/62216
      Change-Id: Ic3a7e0e31a2296aaef121ba6d752c6ef55c041d8
      Reviewed-by: Samuel Rawlins <sraw...@google.com>
      Reviewed-by: Leaf Petersen <le...@google.com>
      Commit-Queue: Kallen Tu <kall...@google.com>
      Files:
      • M CHANGELOG.md
      • M pkg/analysis_server/lib/src/services/completion/dart/feature_computer.dart
      • M pkg/analysis_server/test/services/completion/dart/location/dot_shorthand_constructor_invocation_test.dart
      • M pkg/analysis_server/test/src/services/completion/dart/feature_computer_test.dart
      Change size: M
      Delta: 4 files changed, 60 insertions(+), 1 deletion(-)
      Branch: refs/heads/stable
      Submit Requirements:
      • requirement satisfiedCherry-Pick-Review: Code-Review+1 by Samuel Rawlins, Code-Review+1 by Leaf Petersen
      • requirement satisfiedCode-Review: +1 by Samuel Rawlins, +1 by Leaf Petersen
      Open in Gerrit
      Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
      Gerrit-MessageType: merged
      Gerrit-Project: sdk
      Gerrit-Branch: stable
      Gerrit-Change-Id: Ic3a7e0e31a2296aaef121ba6d752c6ef55c041d8
      Gerrit-Change-Number: 467384
      Gerrit-PatchSet: 3
      Gerrit-Owner: Kallen Tu <kall...@google.com>
      open
      diffy
      satisfied_requirement
      Reply all
      Reply to author
      Forward
      0 new messages