Skip to content

Added .gitignore and fixed handling of HTML encoded characters. - #1

Merged
schmijos merged 3 commits into
schmijos:masterfrom
ypid:optimized
Aug 12, 2015
Merged

Added .gitignore and fixed handling of HTML encoded characters.#1
schmijos merged 3 commits into
schmijos:masterfrom
ypid:optimized

Conversation

@ypid

@ypid ypid commented Aug 3, 2015

Copy link
Copy Markdown
Collaborator

No description provided.

Comment thread .gitignore Outdated

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not a fan of over-specific .gitignore files. Is it ok for you to only have the first 4 lines in it?

Comment thread html_table_parser/parser.py Outdated

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there any reason for that you interpret content of a table cell? I think it's not the purpose of an HTML table parser to do so.

schmijos added a commit that referenced this pull request Aug 12, 2015
Made separator for stripped-out tags configurable. Also make it possible to configure unescaping of html entities.
@schmijos
schmijos merged commit effdef9 into schmijos:master Aug 12, 2015
ypid added a commit to ypid/html-table-parser-python3 that referenced this pull request Aug 12, 2015
…lated to schmijos#1.

* Also renamed the html_table_to_csv_converter to html_table_converter
  to have allow more flexibility for more output formats.
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.

2 participants