-
Notifications
You must be signed in to change notification settings - Fork 0
Created polluted location reading from file #12
Conversation
I haven't looked at the changed files yet, but I can see that this PR has 1 commit. Since this was a pretty big task, try to do incremental commits in the future to get a more gradual flow of updates. Nonetheless, thanks for doing this, I will review it shortly |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall looking good, although please address the comments and push changes if needed. We may need to make the IPollutedLocationRepository
methods async
(database queries will definitely be async), but let's do this one step at a time.
mapping null to exceptions remain.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Small things
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM now, feel free to merge
What was done