Skip to content

Reorganize CI testing. #322

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
Jan 2, 2017
Merged

Reorganize CI testing. #322

merged 1 commit into from
Jan 2, 2017

Conversation

BurntSushi
Copy link
Member

Writing all of the testing scripts inside the .travis.yml file was
becoming painful, and parts of it were wrong by allowing for some
commands to fail without failing the entire build.

This also fixes the Github token (again).

@BurntSushi BurntSushi self-assigned this Jan 2, 2017
@BurntSushi
Copy link
Member Author

@bors r+

@bors
Copy link
Contributor

bors commented Jan 2, 2017

📌 Commit f78f446 has been approved by BurntSushi

@bors
Copy link
Contributor

bors commented Jan 2, 2017

⌛ Testing commit f78f446 with merge 9d14071...

bors added a commit that referenced this pull request Jan 2, 2017
Reorganize CI testing.

Writing all of the testing scripts inside the .travis.yml file was
becoming painful, and parts of it were wrong by allowing for some
commands to fail without failing the entire build.

This also fixes the Github token (again).
@BurntSushi
Copy link
Member Author

@bors r-

@bors
Copy link
Contributor

bors commented Jan 2, 2017

💔 Test failed - status-appveyor

Writing all of the testing scripts inside the .travis.yml file was
becoming painful, and parts of it were wrong by allowing for some
commands to fail without failing the entire build.

This also fixes the Github token (again).
@BurntSushi
Copy link
Member Author

@bors r+

@bors
Copy link
Contributor

bors commented Jan 2, 2017

📌 Commit c7bc06f has been approved by BurntSushi

@bors
Copy link
Contributor

bors commented Jan 2, 2017

⌛ Testing commit c7bc06f with merge a9c52b6...

bors added a commit that referenced this pull request Jan 2, 2017
Reorganize CI testing.

Writing all of the testing scripts inside the .travis.yml file was
becoming painful, and parts of it were wrong by allowing for some
commands to fail without failing the entire build.

This also fixes the Github token (again).
@bors
Copy link
Contributor

bors commented Jan 2, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: BurntSushi
Pushing a9c52b6 to master...

@bors bors merged commit c7bc06f into rust-lang:master Jan 2, 2017
@BurntSushi BurntSushi deleted the organize-ci branch January 2, 2017 22:40
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