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

HACKING_QUICKSTART.md: Don't recommend creating .cargo/config in pare… #9069

Merged
merged 1 commit into from Dec 26, 2015

Conversation

@antrik
Copy link
Contributor

antrik commented Dec 25, 2015

…nt directory

Recommend putting crate overrides directly in the servo/.cargo/config,
rather than in Servo's parent directory.

Overrides in the parent directory are unnecessary and confusing for
typical use cases.

Review on Reviewable

…nt directory

Recommend putting crate overrides directly in the servo/.cargo/config,
rather than in Servo's parent directory.

Overrides in the parent directory are unnecessary and confusing for
typical use cases.
@SimonSapin
Copy link
Member

SimonSapin commented Dec 25, 2015

Thanks!

@bors-servo r+

@nox
Copy link
Member

nox commented Dec 25, 2015

@bors-servo r=SimonSapin rollup

@nox nox closed this Dec 25, 2015
@nox nox reopened this Dec 25, 2015
@nox
Copy link
Member

nox commented Dec 25, 2015

@bors-servo r=SimonSapin rollup

@bors-servo
Copy link
Contributor

bors-servo commented Dec 25, 2015

📌 Commit dac4056 has been approved by SimonSapin

bors-servo added a commit that referenced this pull request Dec 25, 2015
HACKING_QUICKSTART.md: Don't recommend creating .cargo/config in pare…

…nt directory

Recommend putting crate overrides directly in the servo/.cargo/config,
rather than in Servo's parent directory.

Overrides in the parent directory are unnecessary and confusing for
typical use cases.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9069)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Dec 25, 2015

Testing commit dac4056 with merge c6d613f...

@bors-servo
Copy link
Contributor

bors-servo commented Dec 25, 2015

💔 Test failed - linux-rel

@antrik
Copy link
Contributor Author

antrik commented Dec 26, 2015

Yay, documentation tweak broke CSS tests!... Intermittent failures make CI kinda surrealistic ;-)

@nox
Copy link
Member

nox commented Dec 26, 2015

@bors-servo
Copy link
Contributor

bors-servo commented Dec 26, 2015

Previous build results for android, gonk, linux-dev, mac-dev-ref-unit, mac-rel-css, mac-rel-wpt are reusable. Rebuilding only linux-rel...

@nox nox added S-awaiting-merge and removed S-tests-failed labels Dec 26, 2015
@bors-servo
Copy link
Contributor

bors-servo commented Dec 26, 2015

@bors-servo bors-servo merged commit dac4056 into servo:master Dec 26, 2015
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants
You can’t perform that action at this time.