search

urbanplatform / django-keycloak-auth

Middleware to allow authorization using Keycloak and Django for django-rest-framework (DRF). This package should only be used in projects starting from scratch, since it overrides the users' management.
MIT License
32 stars 14 forks source link

Formatting rules #2

Closed moritz89 closed 2 years ago

moritz89 commented 2 years ago

If contributions are welcome, it would be of use to have a set of formatting rules to keep a consistent and automatically formatted code base. Is such a set of rules available?

ftcardoso commented 2 years ago

Hi @moritz89!

Yes, contributions are welcome! Currently there is no set of rules... Any suggestion regarding that?

moritz89 commented 2 years ago

Black is a comprehensive formatter that I've had good experiences with. The most important aspect for me is that there is a standard and I don't have to think about the formatting :smile:

ftcardoso commented 2 years ago

Did a quick test with Black and I think we can go with it! I will run it through the repo and format all the files. We will also check some GitHub action to validate or event format on push/PR. Thanks!

ftcardoso commented 2 years ago

Added action for code validation based on your recommendation.