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

Don't crash if modifying multiple profile files #1040

Merged
merged 1 commit into from Apr 8, 2017

Conversation

brson
Copy link
Contributor

@brson brson commented Apr 8, 2017

Fixes #1034

cc @rmbreak

@brson
Copy link
Contributor Author

brson commented Apr 8, 2017

@bors r=me

@brson
Copy link
Contributor Author

brson commented Apr 8, 2017

@bors r+

@bors
Copy link
Contributor

bors commented Apr 8, 2017

📌 Commit 7e701b1 has been approved by brson

@bors
Copy link
Contributor

bors commented Apr 8, 2017

⌛ Testing commit 7e701b1 with merge 42a9ac3...

bors added a commit that referenced this pull request Apr 8, 2017
Don't crash if modifying multiple profile files

Fixes #1034

cc @rmbreak
@brson brson mentioned this pull request Apr 8, 2017
@smackysnacks
Copy link

LGTM 👍

@bors
Copy link
Contributor

bors commented Apr 8, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: brson
Pushing 42a9ac3 to master...

@bors
Copy link
Contributor

bors commented Apr 8, 2017

👀 Test was successful, but fast-forwarding failed: 422 Update is not a fast forward

@brson brson merged commit 9f92a6d into rust-lang:master Apr 8, 2017
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.

None yet

3 participants