Skip to content

/ servo

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

Avoid UnboundLocalError upon "bogus" usage of ./mach create-wpt #8410

Merged
merged 1 commit into from Nov 8, 2015
Merged

Avoid UnboundLocalError upon "bogus" usage of ./mach create-wpt #8410

merged 1 commit into from Nov 8, 2015

Conversation

@simartin
Copy link
Contributor

simartin commented Nov 8, 2015

Fixes #8407

Review on Reviewable
@simartin
Avoid UnboundLocalError upon "bogus" usage of ./mach create-wpt
Loading status checks…
3b087d4
@nox
Copy link
Member

nox commented Nov 8, 2015

@bors-servo r+
@bors-servo
Copy link
Contributor

bors-servo commented Nov 8, 2015

📌 Commit 3b087d4 has been approved by nox
@bors-servo
Copy link
Contributor

bors-servo commented Nov 8, 2015

⌛ Testing commit 3b087d4 with merge f8623a0...
bors-servo added a commit that referenced this pull request Nov 8, 2015
@bors-servo
Auto merge of #8410 - simartin:issue_8407, r=nox
f8623a0 
Avoid UnboundLocalError upon "bogus" usage of ./mach create-wpt

Fixes #8407

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8410)
<!-- Reviewable:end -->
@nox nox added A-mach L-python and removed S-awaiting-merge labels Nov 8, 2015
@nox nox self-assigned this Nov 8, 2015
@jdm
Copy link
Member

jdm commented Nov 8, 2015

Thanks for figuring this out @simartin!
@bors-servo
Copy link
Contributor

bors-servo commented Nov 8, 2015

💔 Test failed - linux-rel
@frewsxcv
Copy link
Member

frewsxcv commented Nov 8, 2015

@bors-servo retry

#8311
@bors-servo
Copy link
Contributor

bors-servo commented Nov 8, 2015

⌛ Testing commit 3b087d4 with merge 43a5c62...
bors-servo added a commit that referenced this pull request Nov 8, 2015
@bors-servo
Auto merge of #8410 - simartin:issue_8407, r=nox
Loading status checks…
43a5c62 
Avoid UnboundLocalError upon "bogus" usage of ./mach create-wpt

Fixes #8407

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

bors-servo commented Nov 8, 2015

☀️ Test successful - android, gonk, linux-dev, linux-rel, mac-dev-ref-unit, mac-rel-css, mac-rel-wpt
@bors-servo bors-servo merged commit 3b087d4 into servo:master Nov 8, 2015
2 of 3 checks passed
2 of 3 checks passed
code-review/reviewable Review in progress: 0 of 1 files reviewed, all discussions resolved
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
@simartin simartin deleted the simartin:issue_8407 branch Nov 8, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@nox nox

Labels
A-mach L-python S-awaiting-merge
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

6 participants
@simartin @nox @bors-servo @jdm @frewsxcv @highfive
You can’t perform that action at this time.