smart-on-fhir / chart-review

Measure agreement between chart reviewers.
https://docs.smarthealthit.org/cumulus/chart-review/
Apache License 2.0
1 stars 0 forks source link

Various cleanups #5

Closed mikix closed 8 months ago

mikix commented 8 months ago

The above are all different commits for ease of review. I'd look at individual commits rather than the whole PR (but grouping them makes my life easier rather than chained PRs).

Notably absent from the above list is pylint -- mostly because it'd add more noise to this PR. We can always add it later.

mikix commented 8 months ago

BTW I took all your nit suggestions and this is ready to land from my POV, just giving it a bit to breathe in case Andy wants to look in

mikix commented 8 months ago

Eh, I've got another PR I want to file, based on this. I'm going to land now