Skip to content

Require CI success on stable #10

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

Merged
merged 1 commit into from
Feb 24, 2020
Merged

Conversation

demurgos
Copy link
Contributor

This commit removes the allow_failure config for stable Rust. According to the comment it is no longer needed since the release of Rust 1.32.

@eddyb
Copy link
Member

eddyb commented Feb 22, 2020

Let me know when/if you get this working, so I can merge it.

@demurgos
Copy link
Contributor Author

There's an open issue relative to the CI error on nightly: rustwasm/wasm-bindgen#2009.
It seems that the fix is already merged and should be available soon.

@eddyb
Copy link
Member

eddyb commented Feb 23, 2020

Oh I didn't check the error message. Funnily enough I'm aware of that bug because I had to work around it recently in a completely unrelated situation.

This commit removes the `allow_failure` config for stable Rust. According to the comment it is no longer needed since the release of Rust 1.32.
@demurgos
Copy link
Contributor Author

Now that the fixed nightly is available, I triggered CI again and it passed.

@eddyb eddyb merged commit 8c19c63 into LykenSol:master Feb 24, 2020
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