-
Notifications
You must be signed in to change notification settings - Fork 618
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
Implement crates-admin
binary
#3011
Conversation
☔ The latest upstream changes (presumably #2810) made this pull request unmergeable. Please resolve the merge conflicts. Note that reviewers usually do not review pull requests until merge conflicts are resolved! Once you resolve the conflicts, you should change the labels applied by bors to indicate that your PR is ready for review. Post this as a comment to change the labels:
|
The For deployment purposes, |
This will become useful once these structs are used as subcommands
1855141
to
8dddb4e
Compare
@jtgeibel thanks for the review. I've removed |
This description is for `monitor`, which was dropped from this PR.
I pushed a commit dropping the about attribute on I'm fine with migrating Merging this PR and deploying to staging. @bors r+ |
📌 Commit 9aa026c has been approved by |
☀️ Test successful - checks-actions |
This PR resolves #1504 by adding a new
crates-admin
binary, that combines a lot of the previously separate binaries as subcommands.I would recommend to review commit-by-commit :)
r? @jtgeibel