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

Remove support for legacy credentials file #3824

Merged

Conversation

sigurdm
Copy link
Contributor

@sigurdm sigurdm commented Mar 13, 2023

We stopped creating this in dart 2.15

https://github.com/dart-lang/sdk/blob/main/CHANGELOG.md#pub-4

One might say we should support this longer - but I think Dart 3.0 is a good place to make this change. And also the consequence is only the user would have to log in again.

@sigurdm sigurdm closed this Mar 13, 2023
@sigurdm sigurdm reopened this Mar 13, 2023
/// best place for storing secrets, as it might be shared.
///
/// To provide backwards compatibility we use the legacy file if only it exists.
///
/// Returns `null` if there is no good place for the file.
String? _credentialsFile(SystemCache cache) {
Copy link
Member

Choose a reason for hiding this comment

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

We don't need SystemCache as argument

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good catch. Removed

@sigurdm sigurdm merged commit cfaec21 into dart-lang:master Mar 15, 2023
lbguilherme added a commit to sdkgen/sdkgen that referenced this pull request May 18, 2023
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