Closed brittainhard closed 8 years ago
LGTM. My only comment is that in the docker image, I point to scrapinghub for the scrapy install. Since they are the package maintainers, I think that's a little better.
@msarahan going to merge this. I'm going to add an issue to specify to get scrapy from scrapinghub.
@msarahan @youngblood look ok?