Skip to content

Include source location in param of rule parsers #226

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

Merged
merged 1 commit into from
Jun 26, 2018

Conversation

upsuper
Copy link
Contributor

@upsuper upsuper commented Jun 26, 2018

This change is Reviewable

@upsuper
Copy link
Contributor Author

upsuper commented Jun 26, 2018

r? @SimonSapin

@upsuper
Copy link
Contributor Author

upsuper commented Jun 26, 2018

cc servo/servo#20527

@upsuper
Copy link
Contributor Author

upsuper commented Jun 26, 2018

See https://bugzilla.mozilla.org/show_bug.cgi?id=1471104 for the Gecko/Servo side change.

@SimonSapin
Copy link
Member

Looks good.

@bors-servo r+

@bors-servo
Copy link
Contributor

📌 Commit 901eb07 has been approved by SimonSapin

@bors-servo
Copy link
Contributor

⌛ Testing commit 901eb07 with merge 22dfc44...

bors-servo pushed a commit that referenced this pull request Jun 26, 2018
Include source location in param of rule parsers

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

☀️ Test successful - status-travis
Approved by: SimonSapin
Pushing 22dfc44 to master...

@bors-servo bors-servo merged commit 901eb07 into servo:master Jun 26, 2018
@upsuper upsuper deleted the source-location branch June 26, 2018 06:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants