-
Notifications
You must be signed in to change notification settings - Fork 75
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
Update README.md to use dart pub
instead of pub
#2195
Conversation
The `pub` command no longer exist and users are therefore required to use `dart pub` instead for installation.
Thanks! The failing release reminder test should be fixed by merging this branch with master |
Hope I done the pull correctly and everything is okay now. |
Thank you! LGTM |
auto label is removed for dart-lang/webdev/2195, due to This PR has not met approval requirements for merging. You are not a member of dart-team and need 1 more review(s) in order to merge this PR.
|
@elliette Do you know why this pull-request have yet to be merged? It does not really have any kind of deadline but I just find it weird it just stays in this state like it is waiting for something? |
You will need to merge it manually (with the Squash and merge button) - if you don't see it, I can merge it for you. |
Well, I don't have that button so you are allowed to do so for me :) |
The
pub
command no longer exist and users are therefore required to usedart pub
instead for installation.Contribution guidelines:
dart format
.Note that many Dart repos have a weekly cadence for reviewing PRs - please allow for some latency before initial review feedback.