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

Add function to change tokenizer state to Plaintext #183

Merged
merged 1 commit into from Nov 20, 2015

Conversation

@KiChjang
Copy link
Member

KiChjang commented Nov 20, 2015

Fixes #109, I'm really not sure whether I'm doing the right thing.

r? @kmcallister

Review on Reviewable

@jdm
Copy link
Member

jdm commented Nov 20, 2015

Probably r? @nox instead.

@SimonSapin
Copy link
Member

SimonSapin commented Nov 20, 2015

@bors-servo
Copy link
Contributor

bors-servo commented Nov 20, 2015

📌 Commit c4ea0c4 has been approved by SimonSapin

@bors-servo
Copy link
Contributor

bors-servo commented Nov 20, 2015

Testing commit c4ea0c4 with merge e31c574...

bors-servo added a commit that referenced this pull request Nov 20, 2015
Add function to change tokenizer state to Plaintext

Fixes #109, I'm really not sure whether I'm doing the right thing.

r? @kmcallister

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

bors-servo commented Nov 20, 2015

☀️ Test successful - travis

@bors-servo bors-servo merged commit c4ea0c4 into servo:master Nov 20, 2015
1 of 2 checks passed
1 of 2 checks passed
homu Testing commit c4ea0c4 with merge e31c574...
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@KiChjang KiChjang deleted the KiChjang:change-plaintext-state branch Nov 20, 2015
@SimonSapin
Copy link
Member

SimonSapin commented Nov 20, 2015

Thanks! I published v0.2.10 with this. Do you want to fix servo/servo@82c52a7#diff-14d82a613adf6dbeec0ec4415a1d323aR195 as well?

@KiChjang
Copy link
Member Author

KiChjang commented Nov 20, 2015

That's exactly why I did this. I am working on it as we speak.

@SimonSapin
Copy link
Member

SimonSapin commented Nov 20, 2015

👍

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.