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

Always build CI on master #2940

Closed
wants to merge 1 commit into from
Closed

Always build CI on master #2940

wants to merge 1 commit into from

Conversation

Manishearth
Copy link
Member

This switches travis over to always using master.

We can alternatively pin a hash and bump it every time something breaks, but this feels cleaner (albeit less cacheable on the travis side)

needs kennytm/rustup-toolchain-install-master#6 to be released to work

r? @oli-obk

@Manishearth
Copy link
Member Author

Given that clippy now breaks nightlies, there is no other way to CI PRs that unbreak nightly.

@oli-obk
Copy link
Contributor

oli-obk commented Jul 20, 2018

I tried this before and failed: #2551

@Manishearth
Copy link
Member Author

Oh, perfect.

@Manishearth
Copy link
Member Author

Going to see if stuff is fixed now

@oli-obk
Copy link
Contributor

oli-obk commented Jul 20, 2018

ups

I just fixed the prs branch xD

@oli-obk oli-obk force-pushed the travis-toolchain branch from eaf6270 to f226101 Compare July 20, 2018 08:24
@oli-obk
Copy link
Contributor

oli-obk commented Jul 20, 2018

Ok, rebased and fixed travis

@Manishearth
Copy link
Member Author

er, pick one pull request and we'll stick with it, we're overwriting each others' commits here lol

@oli-obk
Copy link
Contributor

oli-obk commented Jul 20, 2018

lol sorry

@Manishearth
Copy link
Member Author

and we can drop the git fudgery, with my PR rustup-toolchain-install-master -n master -f will work 😄

@flip1995 flip1995 deleted the travis-toolchain branch October 28, 2019 08:25
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