Skip to content
This repository was archived by the owner on Nov 6, 2020. It is now read-only.

add --force, it's installed somewhere#9919

Closed
TriplEight wants to merge 1 commit into
masterfrom
fix-cargo-audit
Closed

add --force, it's installed somewhere#9919
TriplEight wants to merge 1 commit into
masterfrom
fix-cargo-audit

Conversation

@TriplEight
Copy link
Copy Markdown
Collaborator

it appeared that cargo audit was installed previously, that failed the beta and stable builds:

cargo-audit v0.5.2
error: binary `cargo-audit` already exists in destination as part of `cargo-audit v0.5.2`
Add --force to overwrite

@TriplEight TriplEight requested review from bkchr and ordian November 14, 2018 12:42
@parity-cla-bot
Copy link
Copy Markdown

It looks like @TriplEight signed our Contributor License Agreement. 👍

Many thanks,

Parity Technologies CLA Bot

@ordian
Copy link
Copy Markdown
Member

ordian commented Nov 14, 2018

I think we don't need it on master, since we've removed caches in #9855, so we either backport this or that PR.

@bkchr
Copy link
Copy Markdown

bkchr commented Nov 14, 2018

I support @ordian opinion on this topic.

@TriplEight TriplEight closed this Nov 14, 2018
@ordian ordian deleted the fix-cargo-audit branch November 14, 2018 13:08
@5chdn 5chdn added this to the 2.3 milestone Nov 25, 2018
@5chdn 5chdn added A5-grumble 🔥 Pull request has minor issues that must be addressed before merging. M1-ci 🙉 Continuous integration. labels Nov 25, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

A5-grumble 🔥 Pull request has minor issues that must be addressed before merging. M1-ci 🙉 Continuous integration.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants