Skip to content
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

Add support for package:web 1.0.0 #619

Merged
merged 1 commit into from
Jul 24, 2024
Merged

Conversation

srujzs
Copy link
Contributor

@srujzs srujzs commented Jul 24, 2024

1.0.0 modifies innerHTML to be JSAny in order to support trusted types, and this is the only member that needs to be worked around to support that version.

1.0.0 modifies innerHTML to be JSAny in order to
support trusted types, and this is the only member
that needs to be worked around to support that version.
@srujzs srujzs requested a review from kevmoo July 24, 2024 21:42
@coveralls
Copy link

Pull Request Test Coverage Report for Build 10084439722

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 96.099%

Totals Coverage Status
Change from base Build 9745355288: 0.0%
Covered Lines: 1552
Relevant Lines: 1615

💛 - Coveralls

@srujzs srujzs merged commit f6eaea3 into dart-lang:master Jul 24, 2024
8 checks passed
copybara-service bot pushed a commit to dart-lang/sdk that referenced this pull request Jul 25, 2024
Revisions updated by `dart tools/rev_sdk_deps.dart`.

markdown (https://github.com/dart-lang/markdown/compare/6242437..f6eaea3):
  f6eaea3  2024-07-24  Srujan Gaddam  Add support for package:web 1.0.0 (dart-lang/markdown#619)

test (https://github.com/dart-lang/test/compare/2c2ac49..d0dc833):
  d0dc8337  2024-07-22  Nate Bosch  Migrate off deprecated Analyzer API (dart-lang/test#2258)

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

3 participants