OpServ-Monitoring / opserv-backend

The backend repository for the opserv monitoring software
GNU General Public License v3.0
3 stars 0 forks source link

Combine REST-API endpoint files #20

Open Laberbear opened 7 years ago

Laberbear commented 7 years ago

The endpoint folder of the REST-API has a file for every endpoint. This currently aggregates to 62 Python files in that directory. While it is advised in the PEP8 coding standard to have one class per file and name the file accordingly, I think this greatly decreases the ability to comprehend the backend structure and editability of the REST-API.

https://github.com/OpServ-Monitoring/opserv-backend/tree/master/app/server/restful_api/data/v1/endpoints

Unless there are serious issues with combining (atleast the components) into bigger Python files, I suggest we do just that.