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

sep 4 for #629 #633

Merged
merged 2 commits into from Mar 7, 2014
Merged

sep 4 for #629 #633

merged 2 commits into from Mar 7, 2014

Conversation

@aspidites
Copy link
Contributor

@aspidites aspidites commented Mar 6, 2014

No description provided.

for implementing crawlers by just using callback functions. That way you could
use Scrapy as any standard library (like you would use os.walk) in a script
without the overhead of having to create an entire project from scratch.

This comment has been minimized.

@aspidites

aspidites Mar 6, 2014
Author Contributor

I removed the following lines as the snippets were no longer there:

See these snippets for some examples:
     * http://snippets.scrapy.org/snippets/8/
     * http://snippets.scrapy.org/snippets/9/

Should I re-add them for historical accuracy?

This comment has been minimized.

@pablohoffman

pablohoffman Mar 7, 2014
Member

maybe they were moved to snipplr? http://snipplr.com/all/tags/scrapy/

This comment has been minimized.

@aspidites

aspidites Mar 7, 2014
Author Contributor

Without the originals I'm not sure which of those it would be and as I'm new to scrapy myself, I'm not inclined to guess, especially if the API has changed slightly as suggested by the SEP.

pablohoffman added a commit that referenced this pull request Mar 7, 2014
sep 4 for #629
@pablohoffman pablohoffman merged commit 690fbdb into scrapy:master Mar 7, 2014
1 check passed
1 check passed
default The Travis CI build passed
Details
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

2 participants
You can’t perform that action at this time.