Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Decide on (and convert to) indentation style #5

Open
CFSworks opened this issue Jun 16, 2017 · 3 comments
Open

Decide on (and convert to) indentation style #5

CFSworks opened this issue Jun 16, 2017 · 3 comments

Comments

@CFSworks
Copy link
Contributor

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
Copy link
Collaborator

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

@zahodi
Copy link
Owner

zahodi commented Jun 16, 2017

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
Copy link
Collaborator

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants