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

Prepare InstallPluginCommand for use with Dart 2.19 (default pub hosted url changed) #221

Merged
merged 2 commits into from Jan 31, 2023

Conversation

Giuspepe
Copy link
Contributor

In Dart 2.19, the default pub-cache folder changed from .pub-cache/pub.dartlang.org to .pub-cache/pub.dev. Without this change, InstallPluginCommand would break because we would be looking for the downloaded plugin in the wrong cache folder.

most important change is that the default pub-cache folder changes from `.pub-cache/pub.dartlang.org` to `.pub-cache/pub.dev`. Without this change, InstallPluginCommand would break
@Giuspepe Giuspepe requested a review from passsy January 31, 2023 12:19
/// We change `https://pub.dev` to `https://pub.dartlang.org`, this maintains
/// avoids churn for `pubspec.lock`-files which contain
/// `https://pub.dartlang.org`.
/// Else:
/// We change `https://pub.dartlang.org` to `https://pub.dev`, this maintains
/// backwards compatibility with `pubspec.lock`-files which contain
Copy link
Contributor

Choose a reason for hiding this comment

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

backwards compatibility 🎉

@passsy passsy merged commit e97f192 into main Jan 31, 2023
@passsy passsy deleted the prepare-pub-dart-2-19 branch January 31, 2023 12:55
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.

None yet

2 participants