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

Allow reftests to be run directly w/ relative path #4053

Closed
wants to merge 1 commit into from

Conversation

@mttr
Copy link
Contributor

mttr commented Nov 21, 2014

Fixes #3521

@hoppipolla-critic-bot
Copy link

hoppipolla-critic-bot commented Nov 21, 2014

Critic review: https://critic.hoppipolla.co.uk/r/3234

This is an external review system which you may optionally use for the code review of your pull request.

In order to help critic track your changes, please do not make in-place history rewrites (e.g. via git rebase -i or git commit --amend) when updating this pull request.

@SimonSapin

This comment has been minimized.

Copy link

SimonSapin commented on 3149eae Nov 21, 2014

r+

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented on 3149eae Nov 21, 2014

saw approval from SimonSapin
at mttr@3149eae

This comment has been minimized.

Copy link
Contributor

bors-servo replied Nov 21, 2014

merging mttr/servo/reftests_with_relative_dir = 3149eae into auto

This comment has been minimized.

Copy link
Contributor

bors-servo replied Nov 21, 2014

mttr/servo/reftests_with_relative_dir = 3149eae merged ok, testing candidate = dfa9e1a

This comment has been minimized.

Copy link
Contributor

bors-servo replied Nov 21, 2014

fast-forwarding master to auto = dfa9e1a

bors-servo pushed a commit that referenced this pull request Nov 21, 2014
@bors-servo bors-servo closed this Nov 21, 2014
@mttr mttr deleted the mttr:reftests_with_relative_dir branch Nov 21, 2014
@mttr mttr restored the mttr:reftests_with_relative_dir branch Nov 21, 2014
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.

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