Apply aggregate to location

ExpDev07 / coronavirus-tracker-api

🦠 A simple and fast (< 200ms) API for tracking the global coronavirus (COVID-19, SARS-CoV-2) outbreak. It's written in python using the 🔥 FastAPI framework. Supports multiple sources!

https://coronavirus-tracker-api.herokuapp.com

GNU General Public License v3.0

1.59k stars 323 forks source link

Apply aggregate to location #359

Closed wu-geoff closed 3 years ago

wu-geoff commented 3 years ago

Turning most of the variables into protected, because there is no point making them public. Appropriate getters are added. PS: It passed all the unit tests.

Kilo59 commented 3 years ago

Apologies but I don't see the value in adding 50+ new lines of code just to make these attributes "private". What are we protecting against here?