Skip to content
This repository has been archived by the owner on Dec 29, 2021. It is now read-only.

No unnecessary rebuilds on CI #52

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

No unnecessary rebuilds on CI #52

wants to merge 1 commit into from

Conversation

killercup
Copy link
Collaborator

Why would we need to build, clean, build again? I guess either the
author of this commit or of the author of 1e04c94 was drunk or
something.

Why would we need to build, clean, build again? I guess either the
author of this commit or of the author of 1e04c94 was drunk or
something.
@epage
Copy link
Collaborator

epage commented Oct 13, 2017

I know I've run into bugs with Skeptic where I have to cargo clean for the tests to not die with errors about multiple options of <NAME> crate.

Similar error as budziq/rust-skeptic#27, not sure if root cause is the same.

@epage
Copy link
Collaborator

epage commented Oct 13, 2017

btw in my other projects I switched to cargo check, assuming there is no loss of validation from it

@colin-kiegel
Copy link
Collaborator

@epage that sounds more like budziq/rust-skeptic#18 and budziq/rust-skeptic#37 which are marked as solved since July 12th. But I haven't checked that myself. If you can reproduce it, you could maybe reopen one of these issues.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants