FEM Python coding standard

[Kunda1]

@bernd, when i type codespell -L:

Code: Select all

codespell: error: no such option: -L

Code: Select all

 -L WORDS, --ignore-words-list WORDS
                        Comma separated list of words to be ignored by
                        codespell. Words are case sensitive based on how they
                        are written in the dictionary file

From codespell --help

[ebrahim raeyat]

Code: Select all

  -L WORDS, --ignore-words-list WORDS

when i type this command with -L flag, it takes me this error:

Code: Select all

Usage: 
	codespell [OPTIONS] [file1 file2 ... fileN]


codespell: error: no such option: -L

codespell version 1.8

[Kunda1]

codespell version 1.8

Code: Select all

$ codespell --version
1.17.0.dev0

Check out https://github.com/codespell-project/codespell#updating

[ebrahim raeyat]

codespell version 1.8

Code: Select all

$ codespell --version
1.17.0.dev0

Check out https://github.com/codespell-project/codespell#updating

thanks, it works.

[ebrahim raeyat]

double posted! Is solved here https://forum.freecadweb.org/viewtopic. ... 70#p380760

yes, i prefer to solve this in related topic. if you can please move those to related topic. thanks.

[vanuan]

See my formal proposal: https://github.com/FreeCAD/FreeCAD-Enha ... s/issues/9

[bernd]

are they conform with FEM conding standards? See https://github.com/FreeCAD/FreeCAD/blob ... entions.md

[vanuan]

are they conform with FEM conding standards? See https://github.com/FreeCAD/FreeCAD/blob ... entions.md

Have you read the proposal? It mainly focuses on a method to enforce those. But yeah, the first rules I propose to enable are the snake_case naming and the import grouping which are included in the document you referenced.

Does FEM already use flake8?

[bernd]

are they conform with FEM conding standards? See https://github.com/FreeCAD/FreeCAD/blob ... entions.md

Have you read the proposal? It mainly focuses on a method to enforce those. But yeah, the first rules I propose to enable are the snake_case naming and the import grouping which are included in the document you referenced.

Does FEM already use flake8?

the document I referenced only has coding standards which are fulfilled by FEM python code already. I only add new coding standard if at least the most code of FEM does use the coding standard. Thus there is only rare coding standard for FEM C++ code, since I do not do that much c++ code and thus I do not change the coding standard.

[bernd]

AFAIK FEM and Draft uses flake8. Only difference is the indent. Draft uses other indent style as FEM.

vocx might give some more information in this regard.