Skip to content

allow custom content_extractore - #152

Closed
itech001 wants to merge 2 commits into
codelucas:masterfrom
itech001:master
Closed

allow custom content_extractore#152
itech001 wants to merge 2 commits into
codelucas:masterfrom
itech001:master

Conversation

@itech001

Copy link
Copy Markdown

the default content_extractor doesn't work for me, i need have one custom content extractor to work for my special task

Comment thread newspaper/__init__.py

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Why add imports here?

@yprez

yprez commented Jan 18, 2016

Copy link
Copy Markdown
Collaborator

Why not set it outside of the library instead?
e.g.

article = Article()
article.extractor = CustomExtractor()

article.download()
#...

@codelucas codelucas closed this Jan 30, 2016
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