Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign upImplement HTMLOptionElement.text #3023 #3364
Conversation
hoppipolla-critic-bot
commented
Sep 16, 2014
|
Critic review: https://critic.hoppipolla.co.uk/r/2606 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 |
highfive
commented
Sep 16, 2014
|
Thanks for the pull request, and welcome! The Servo team is excited to review your changes, and you should hear from @kmcallister (or someone else) soon. |
|
I have left comments on Critic. When addressing them, please make additional commits instead of rewriting the one that has been reviewed. Thanks for the patch! |
aydinkim
commented
Sep 19, 2014
|
Yeah, we keep having a broken master; sorry about that. @jejuliekim can you please squash these commits together and push again? |
704f598
to
5c1fa49
|
I'll look into it, why it causes TC failed. |
|
@jdm , Do I need to upload new commit with removing skip-metadata for option-text here? |
|
Please squash the commits together and push it in this PR. |
Signed-off-by: Jeongeun Kim <je00julie.kim@gmail.com> Implement HTMLOptionElement.text#3023 (2nd trial) Implement HTMLOptionElement.text#3023 (3rd trial) delete metadata for skipping option tests fix conflict
jkim-julie commentedSep 16, 2014
Implement HTMLOptionElement.text