Closed MoralCode closed 3 years ago
Can you rebase to the latest and remove your changes to poetry lock in this PR?
@bryanculbertson yeeah that makes sense. I did best to remove the unnecessary changes to the lockfiles, but this branch requires an update that i made in the latest version of my parse-opening-hours
library that it needs to function and poetry seems to keep pulling in additional updates when installing it :/ any suggestions for how to proceed?
@MoralCode Ah, I see. Lets upgrade parse-opening-hours
separately. I would like all packages updates to go via dependabot if we can as a way of tracking them
@MoralCode Processing the library update in this PR: https://github.com/CAVaccineInventory/vaccine-feed-ingest/pull/819
ah, did not realize dependabot did normal updates and not just security updates. neat
@MoralCode Library update has processed. You can now rebase this PR.
@bryanculbertson finally done + rebased. had to make some changes to the vaccine type detection since the initial version was only set for pfizer so that may need a review since it was added just now.
other than that all is rebased and the commits that change the lockfile are totally gone
Normalize tx/harriscounty_gov
Notes
Seems pretty standard. not much to report. the vaccine inventory normalizer might break if it sees any value besides
Moderna
but it seems like all the sites so far only carry Moderna so that should be fine for nowData sample
Before Opening a PR
poetry run vaccine-feed-ingest <state>/<site>
)poetry run tox -e lint-fix