biglocalnews / warn-scraper

Command-line interface for downloading WARN Act notices of qualified plant closings and mass layoffs from state government websites
https://warn-scraper.readthedocs.io
Apache License 2.0
29 stars 10 forks source link

Hi scraper #371 #497

Closed Ash1R closed 1 year ago

Ash1R commented 2 years ago

For issue #371, added a scraper for HI data. It was a relatively simple list of pdf links, the instructions were to simply get the links since the format inside was too irregular.

stucka commented 1 year ago

Going to see if we can get a fresh code review by closing and reopening.

stucka commented 1 year ago

Looks like this may require a few more things to clear builds and tests: types-python-dateutil = "" python-dateutil = ""

I'll plan to evaluate it and see if dateutil is actually needed or if we can work without it.

pipenv generally does not like me adding stuff, and this rolled in with 300 lines deleted and 300 lines added.

stucka commented 1 year ago

Good and bad news here:

stucka commented 1 year ago

@Ash1R , can you review?

stucka commented 1 year ago

Linting changes seem to have stuck this time. I'm backing away slowly without making eye contact.

stucka commented 1 year ago

Thank you, @Ash1R . And thank you!