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

deny compiler warnings in CI #2570

Merged
merged 1 commit into from Oct 11, 2016
Merged

deny compiler warnings in CI #2570

merged 1 commit into from Oct 11, 2016

Conversation

rphmeier
Copy link
Contributor

(not sure if this qualifies as M-Core)

@rphmeier rphmeier added the A0-pleasereview 🤓 Pull request needs code review. label Oct 11, 2016
@@ -264,7 +265,7 @@ windows:
- set INCLUDE=C:\Program Files (x86)\Microsoft SDKs\Windows\v7.1A\Include;C:\vs2015\VC\include;C:\Program Files (x86)\Windows Kits\10\Include\10.0.10240.0\ucrt
- set LIB=C:\vs2015\VC\lib;C:\Program Files (x86)\Windows Kits\10\Lib\10.0.10240.0\ucrt\x64
- set RUST_BACKTRACE=1
- set RUSTFLAGS=-Zorbit=off
- set RUSTFLAGS=%RUSTFLAGS% -Zorbit=off
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this right? can't really test on windows :)

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks right to me

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 749f64b on deny-warn-ci into * on master*.

@arkpar arkpar added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Oct 11, 2016
@arkpar arkpar merged commit 303654d into master Oct 11, 2016
@gavofyork gavofyork deleted the deny-warn-ci branch November 3, 2016 11:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants