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

moving to travis crates infrastructure #322

Merged
merged 2 commits into from
Oct 27, 2015
Merged

moving to travis crates infrastructure #322

merged 2 commits into from
Oct 27, 2015

Conversation

Vinatorul
Copy link
Contributor

The idea is to get rid of sudo in the script, so travis can run builds on new crates infrastructure. It will increase build speed and decrease load on travis.

@yo-bot
Copy link

yo-bot commented Oct 27, 2015

r? @james-darkfox

(yo-bot has picked a reviewer for you, use r? to override)

@Vinatorul Vinatorul changed the title chore: stop using sudo moving to travis crates infrastructure Oct 27, 2015
@Vinatorul
Copy link
Contributor Author

Btw, we will not see the time change in the log, thats why new way only prevent travis from booting virtual machines for each build. The time savings achieved due to that build will start immediatelly.

@kbknapp
Copy link
Member

kbknapp commented Oct 27, 2015

awesome :)

@homu r+

@homu
Copy link
Contributor

homu commented Oct 27, 2015

📌 Commit 6f186a5 has been approved by kbknapp

@homu
Copy link
Contributor

homu commented Oct 27, 2015

⌛ Testing commit 6f186a5 with merge 3fcdc6f...

homu added a commit that referenced this pull request Oct 27, 2015
moving to travis crates infrastructure

The idea is to get rid of `sudo` in the script, so `travis` can run builds on new crates infrastructure. It will increase build speed and decrease load on `travis`.
@homu
Copy link
Contributor

homu commented Oct 27, 2015

☀️ Test successful - status

@homu homu merged commit 6f186a5 into master Oct 27, 2015
@Vinatorul Vinatorul deleted the dev-vin branch October 27, 2015 14:42
@kbknapp kbknapp mentioned this pull request Oct 28, 2015
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.

5 participants