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

Add --token argument #197

Merged
merged 2 commits into from
Dec 20, 2019
Merged

Add --token argument #197

merged 2 commits into from
Dec 20, 2019

Conversation

Jake-Shadle
Copy link
Contributor

Currently, the cargo publish step uses whichever token is in the users ~/.cargo/credentials, or can be overriden by CARGO_REGISTRY_TOKEN (for crates.io), but it is sometimes convenient to specify the --token parameter for cargo publish, which this PR adds.

@sunng87
Copy link
Collaborator

sunng87 commented Dec 20, 2019

LGTM

@sunng87 sunng87 merged commit c3977df into crate-ci:master Dec 20, 2019
epage added a commit to epage/cargo-release that referenced this pull request Jun 3, 2021
Looks like this was broken in crate-ci#197 when we made passing arguments to
`cargo publish` more flexible.

Fixes crate-ci#259
epage added a commit to epage/cargo-release that referenced this pull request Jun 3, 2021
Looks like this was broken in crate-ci#197 when we made passing arguments to
`cargo publish` more flexible.

Fixes crate-ci#259
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