search

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

Fixed MI issue #469 . First commit to this project #495

Open Ash1R opened 2 years ago

Ash1R commented 2 years ago

This is for issue #469 , my first commit to this project. In summary, there wasn't an easily extractable table, so I had to use the space between columns in each row. There were two categories of pdfs: 2000-2006 and 2007-2015 which had to be processed differently. Each had its own set of rows with irregular spacing that had to be handled in different ways.

palewire commented 2 years ago

Thanks for the patch! Is there a good time we could get on Zoom together and look at this?

Ash1R commented 2 years ago

Sure! I can meet today (Wed Aug 17) at any time, tomorrow and day after tomorrow after 5:30, or any time on the weekend.

Ash1R commented 2 years ago

Hi Ben,

Thanks for looking at my patch! I can meet anytime on the weekend, after 3:30 on Tues-Thurs next week, or after 5:15 on Monday and Friday.

Best, Ashir

On Tue, Aug 16, 2022 at 11:49 AM Ben Welsh @.***> wrote:

Thanks for the patch! Is there a good time we could get on Zoom together and look at this?

— Reply to this email directly, view it on GitHub https://github.com/biglocalnews/warn-scraper/pull/495#issuecomment-1217018306, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQPD7FDGDFLT5FVVFRJECILVZPPDBANCNFSM55W266SQ . You are receiving this because you authored the thread.Message ID: @.***>

Ash1R commented 1 year ago

I saw that you wanted some changes on the Michigan scraper, so I made them in the correct PR this time