|
|
- [Git Practices](git-practices)
|
|
- [Git Practices](git-practices)
|
|
|
|
|
|
|
|
|
### Contributing to Sverchok
|
|
|
|
|
|
|
|
Please follow our coding conventions. The simpler your contribution is, the easier it is to follow our conventions. If you are contributing something larger, look at the larger existing nodes for suggestions on formatting and naming conventions.
|
|
|
|
|
|
|
|
#### pep-8
|
|
|
|
|
|
|
|
We follow pep8 mostly. Line-length however is not something we obsess about. Make your lines as long as you need. We also ignore smaller style issues. If you use PyLint, these are some of the ignores you'll find in the codebase:
|
|
|
|
|
|
|
|
```python
|
|
|
|
# pylint: disable=w0122
|
|
|
|
# pylint: disable=w0123
|
|
|
|
# pylint: disable=w0141
|
|
|
|
# pylint: disable=w0613
|
|
|
|
# pylint: disable=C0326
|
|
|
|
```
|
|
|
|
|
|
|
|
What do those mean?
|
|
|
|
|
|
|
|
| code | meaning | controversial? |
|
|
|
|
|------|---------|----------------|
|
|
|
|
|w0122 | Use of exec | depends, we will try to provide an alternative if possible |
|
|
|
|
|w0123 | eval-used | depends, we will try to provide an alternative if possible |
|
|
|
|
|w0141 | Used builtin function | this means you might have used `map` or `filter` |
|
|
|
|
|w0613 | unused-argument | often the context argmenent in draw_buttons is unused.. |
|
|
|
|
|C0326 | bad-whitespace | sometimes whitespace in comma separated content is a bit annoying.. | |
