Skip to content

Only consider extension types that start with JS as JS types #224

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

Merged
merged 1 commit into from
Apr 12, 2024

Conversation

srujzs
Copy link
Contributor

@srujzs srujzs commented Apr 12, 2024

ExternalDartReference has been added to dart:js_interop but is not a JS type and doesn't belong in the type hierarchy for union calculation. Fixes an assertion failure.

ExternalDartReference has been added to dart:js_interop but is
not a JS type and doesn't belong in the type hierarchy for union
calculation.
@srujzs srujzs requested a review from kevmoo April 12, 2024 18:47
@srujzs srujzs merged commit 9d8c802 into dart-lang:main Apr 12, 2024
copybara-service bot pushed a commit to dart-lang/sdk that referenced this pull request Apr 15, 2024
Revisions updated by `dart tools/rev_sdk_deps.dart`.

dartdoc (https://github.com/dart-lang/dartdoc/compare/ba77660..f339b09):
  f339b094  2024-04-12  Sam Rawlins  Properly sort unnamed libraries (dart-lang/dartdoc#3746)

ecosystem (https://github.com/dart-lang/ecosystem/compare/95fed18..9fabe46):
  9fabe46  2024-04-15  Moritz  Add check for API leaks to the Health workflow (dart-lang/ecosystem#251)

web (https://github.com/dart-lang/web/compare/3d1b4cb..9d8c802):
  9d8c802  2024-04-12  Srujan Gaddam  Only consider extension types that start with JS as JS types (dart-lang/web#224)
  b53bd7f  2024-04-12  M. Hanif Shahy  Update README.md (dart-lang/web#223)

Change-Id: I1009a9acb7fbbf6f276139f1e1a4a5ae2b1e16f5
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/362822
Reviewed-by: Konstantin Shcheglov <[email protected]>
Commit-Queue: Konstantin Shcheglov <[email protected]>
Auto-Submit: Devon Carew <[email protected]>
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.

2 participants