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

Python #27

Open
ax3l opened this issue Dec 5, 2017 · 1 comment
Open

Python #27

ax3l opened this issue Dec 5, 2017 · 1 comment

Comments

@ax3l
Copy link
Member

ax3l commented Dec 5, 2017

For Python scripts, PIConGPU currently enforces:

Versions

Both support for Python 2.7+ and Python 3

Style (PEP8 conformance)

pip install -U flake8
flake8 .

in both Python 2 & Python 3 variants of the flake8 script.
(Note: Recognizes only python scripts ending on *.py, unfortunately.)

Style (Comments)

propose: numpy style
tool: ...?

Static Code Analysis (Warnings, unused code, etc.)

pip install -U pyflakes
pyflakes .

in both Python 2 & Python 3 variants of the flake8 script.
(Note: Recognizes all python scripts.)

@ax3l
Copy link
Member Author

ax3l commented Dec 5, 2017

I would also be personally be a big fan of dropping Python 2 support altogether. Anyone against this?

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

No branches or pull requests

1 participant