Skip to content

Revert "Fixed scheduler process_spider_output() to yield requests"#273

Closed
voith wants to merge 1 commit intoscrapinghub:masterfrom
voith:revert-254-fix-frontera-scheduler
Closed

Revert "Fixed scheduler process_spider_output() to yield requests"#273
voith wants to merge 1 commit intoscrapinghub:masterfrom
voith:revert-254-fix-frontera-scheduler

Conversation

@voith
Copy link
Copy Markdown
Contributor

@voith voith commented Apr 19, 2017

Reverts #254
In accordance to #254 (comment)

@codecov-io
Copy link
Copy Markdown

codecov-io commented Apr 19, 2017

Codecov Report

Merging #273 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #273   +/-   ##
=======================================
  Coverage   70.09%   70.09%           
=======================================
  Files          68       68           
  Lines        4721     4721           
  Branches      633      633           
=======================================
  Hits         3309     3309           
  Misses       1272     1272           
  Partials      140      140
Impacted Files Coverage Δ
frontera/contrib/scrapy/schedulers/frontier.py 96.69% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a5dd4ab...46d0a9b. Read the comment docs.

@sibiryakov
Copy link
Copy Markdown
Member

This is not needed, let's better fix request enqueueing.

@sibiryakov
Copy link
Copy Markdown
Member

see this #261

@voith voith closed this Apr 24, 2017
@voith voith deleted the revert-254-fix-frontera-scheduler branch April 24, 2017 17:36
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