Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Conversation

@devoncarew
Copy link
Contributor

This is a re-land of the reverted PR, #55916 (it depended on a dart roll that was reverted, and has subsequently been re-landed).

From the original PR:

The SoT repository for package:file has changed from google/file.dart to dart-lang/tools; this PR updates the DEPS file and the hard-coded paths in the two pubspec.yaml files (see dart-lang/tools#674).

@github-actions github-actions bot added the platform-web Code specifically for the web engine label Oct 17, 2024
@devoncarew devoncarew requested a review from zanderso October 18, 2024 00:05
Copy link
Member

@zanderso zanderso left a comment

Choose a reason for hiding this comment

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

This should wait until the internal roll succeeds since we may need to roll back again.

@devoncarew
Copy link
Contributor Author

This should wait until the internal roll succeeds since we may need to roll back again.

Gotcha - I'll hold off on landing this for a bit.

@devoncarew devoncarew requested a review from zanderso October 18, 2024 14:54
Copy link
Member

@zanderso zanderso left a comment

Choose a reason for hiding this comment

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

This is good to go

@devoncarew devoncarew merged commit de569ba into main Oct 21, 2024
@devoncarew devoncarew deleted the revert-55916-revert_066a3c71187bb3b6ef694decb9c32287bcd06e45 branch October 21, 2024 17:39
nick9822 pushed a commit to nick9822/flutter that referenced this pull request Dec 18, 2024
…flutter/engine#55935)

This is a re-land of the reverted PR, flutter/engine#55916 (it depended
on a dart roll that was reverted, and has subsequently been re-landed).

From the original PR:

> The SoT repository for `package:file` has changed from
google/file.dart to dart-lang/tools; this PR updates the DEPS file and
the hard-coded paths in the two pubspec.yaml files (see
dart-lang/tools#674).
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

platform-web Code specifically for the web engine

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants