PEP 8 Changes ToDo

[sveneberth]

Besides #435 there are much more changes necessary to be mostly PEP 8 compliant.

This should be a kind of ToDo and space to discuss this. Especially to find an appropriate time. I generally agree with @tsteinruecken's opinion to do this all at once, but not necessarily now.

• Make all class names CapitalizedWords
  • PEP8-compliant naming of Bone classes #435
• Make variable and method names lower_case_with_underscores
• Rename arguments, variables and attribute names which are python keywords
  • ...Bone.type
  • NumericBone.min/max
  • bytes in core.render.html.env.viur.fileSize
  • type in viur.core.render.html.default.Render.copySuccess
• Reduce line length (tbh 79 is sometimes really short, but lines like this are definitely too long.)
• Use spaces (Use 4-spaces instead of tabs for PEP8-conformity #470)
• Replace assert statements with proper exceptions/conditions or use them only sparingly for internal checks/assumptions. Described in Improving SpatialBone bounds checking and use float instead of int #507 (comment)
• Create a ViUR style guide to define rules which aren't strict regulated in the PEP-8 or deviate.
• Enforce the style by a linter with GitHub-Actions or git-hooks
• Replace sth. like if len(kwargs) == 0: with if not kwargs:

Feel free to edit this list above.

[phorward]

#470 replaces tabs by spaces.

[phorward]

Meanwhile the refactored parts of ViUR are still converted to PEP-8.
Therefore, this issue remains open, to be resolved by ViUR 4.0