Removing unused lines, allowing for explicit interrupt of scrape.

rcackerman / parole-hearing-data

http://www.parolehearingdata.org/

21 stars 12 forks source link

Removing unused lines, allowing for explicit interrupt of scrape. #22

Closed talos closed 9 years ago

talos commented 9 years ago

Removed an unused import and unused iterator variable
Added except for KeyboardInterrupt to allow for graceful stop of scrape partway through.

Ideally, we shouldn't do a general except at all, as it could hide errors that we need to fix. What exception is raised when there is a missing detail page or no results?

rcackerman commented 9 years ago

This looks really good! I'll take a closer look and merge tomorrow.