Skip to content

/ saltfs

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

Vagrantfile improvements #150

Merged
merged 3 commits into from Nov 4, 2015
Merged

Vagrantfile improvements #150

merged 3 commits into from Nov 4, 2015
+13 −3

Conversation

@aneeshusa
Copy link
Member

aneeshusa commented Nov 4, 2015

Review on Reviewable
aneeshusa added 3 commits Nov 4, 2015
@aneeshusa
Use vagrant-cachier to speed up package installs
54cec62
@aneeshusa
Use file-relative paths to allow usage from subdirs
Loading status checks…
a6aacf2
@aneeshusa
Add .vagrant/ to .gitignore
Loading status checks…
fc596ca
@metajack
Copy link
Contributor

metajack commented Nov 4, 2015

@bors-servo r+

Reviewed 1 of 1 files at r1, 1 of 1 files at r2, 1 of 1 files at r3.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from the review on Reviewable.io
@bors-servo
Copy link
Contributor

bors-servo commented Nov 4, 2015

📌 Commit fc596ca has been approved by metajack
bors-servo added a commit that referenced this pull request Nov 4, 2015
@bors-servo
Auto merge of #150 - aneeshusa:vagrantfile-improvements, r=metajack
Loading status checks…
f324681 
Vagrantfile improvements

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

bors-servo commented Nov 4, 2015

⌛ Testing commit fc596ca with merge f324681...
@bors-servo
Copy link
Contributor

bors-servo commented Nov 4, 2015

☀️ Test successful - travis
@bors-servo bors-servo merged commit fc596ca into servo:master Nov 4, 2015
3 checks passed
3 checks passed
code-review/reviewable Review complete: all files reviewed, all discussions resolved
Details
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
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
@aneeshusa @metajack @bors-servo
You can’t perform that action at this time.