search

zahodi / ansible-mikrotik

Mikrotik module collection for ansible
Apache License 2.0
96 stars 33 forks source link

Decide on (and convert to) indentation style #5

Open CFSworks opened 7 years ago

CFSworks commented 7 years ago

Some Python files in the repository are in two-spaces-per-level indentation, while PEP-8 (and Ansible as a whole) uses four-spaces-per-level. Which style should be used for this repository?

senorsmile commented 7 years ago

I was complaining about this exact same problem literally hours ago.

zahodi commented 7 years ago

I personally prefer two-spaces indentation. Didn't really think about Ansible code as a whole which will probably matter if we try to bring it to the core.

senorsmile commented 7 years ago

Since our ultimate goal is to get this in upstream Ansible, we'll want to follow whatever their style guide is.