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.

Sign up for GitHub

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

Jump to bottom

Refactor read_html to use TextParser #4697

Closed
cpcloud opened this issue Aug 27, 2013 · 3 comments · Fixed by #4770
Closed

Refactor read_html to use TextParser #4697

cpcloud opened this issue Aug 27, 2013 · 3 comments · Fixed by #4770
Assignees
@cpcloud
Labels
IO Data IO issues that don't fit into a more specific label IO HTML read_html, to_html, Styler.apply, Styler.applymap Refactor Internal refactoring of code
Milestone
0.13

Comments

@cpcloud
Copy link
Member

cpcloud commented Aug 27, 2013

related: #4679
@cpcloud
Copy link
Member Author

cpcloud commented Aug 28, 2013

@cancan101 Before you put any work into this just wanted to let you know that I've almost got a PR for this ready, so hold off on it. Thanks.

@ghost ghost assigned cpcloud Aug 28, 2013
@cancan101 cancan101 mentioned this issue Sep 18, 2013
Merged
1 task
@cancan101
Copy link
Contributor

@cpcloud Do you think this PR will make v0.13?

@cpcloud
Copy link
Member Author

cpcloud commented Sep 28, 2013

yes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cpcloud cpcloud

Labels
IO Data IO issues that don't fit into a more specific label IO HTML read_html, to_html, Styler.apply, Styler.applymap Refactor Internal refactoring of code
Projects
None yet
Milestone
0.13
Development

Successfully merging a pull request may close this issue.

REF/BUG/ENH/API: refactor read_html to use TextParser
2 participants
@cancan101 @cpcloud