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

Revert the DynamicServoFactory stuff #515

Merged
merged 1 commit into from Oct 20, 2016

Conversation

@larsbergstrom
Copy link
Contributor

larsbergstrom commented Oct 20, 2016

r? @aneeshusa

I need to just revert it all until we can sort out what's going on.


This change is Reviewable

@aneeshusa
Copy link
Member

aneeshusa commented Oct 20, 2016

Can you leave the "Fix steps reading on windows" commit for now? I want to spin up a windows-dev-yaml test builder to get it working.

This reverts commit 4075ac4.
@larsbergstrom larsbergstrom force-pushed the larsbergstrom:revert_stuff branch from de05332 to ca53f31 Oct 20, 2016
@larsbergstrom
Copy link
Contributor Author

larsbergstrom commented Oct 20, 2016

Done!

@aneeshusa
Copy link
Member

aneeshusa commented Oct 20, 2016

@bors-servo r+

I have some ideas about the new Windows failures - expect more PRs soon.

@bors-servo
Copy link
Contributor

bors-servo commented Oct 20, 2016

📌 Commit ca53f31 has been approved by aneeshusa

@bors-servo
Copy link
Contributor

bors-servo commented Oct 20, 2016

Testing commit ca53f31 with merge 2a6d627...

bors-servo added a commit that referenced this pull request Oct 20, 2016
Revert the DynamicServoFactory stuff

r? @aneeshusa

I need to just revert it all until we can sort out what's going on.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/saltfs/515)
<!-- Reviewable:end -->
@larsbergstrom
Copy link
Contributor Author

larsbergstrom commented Oct 20, 2016

Passed on all but mac; merging :-)

@larsbergstrom larsbergstrom merged commit 49faa9d into servo:master Oct 20, 2016
0 of 2 checks passed
0 of 2 checks passed
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
homu Testing commit ca53f31 with merge 2a6d627...
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

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