shreyastheone / buggernaut-backend Goto Github PK

The codebase has quite a few cool features, but the coding style doesn't follow PEP8 practices very well. The few points that I noticed were,

1. Inconsistency in naming - In the codebase somewhere it is followed camelCase and at some places snake_case naming is followed, we can probably make that consistent maybe? 🙂 And snake_case naming would be preferred 😛
2. Number of characters on a line, I think we should keep a max number of characters per-line limit to be 80 (or is it 81?). When calling/declaring a function, if the function call goes beyond this character limit, just break it into two/more lines.
   e.g models.ForeignKey(settings.AUTH_USER_MODEL, related_name="issue_reported_by_user", on_delete=models.CASCADE)
   could be further broken into seperate lines. Same goes for lists and dicts used
3. Leave two lines between your different class/function declarations. 😛