Rafael Guterres Jeffman
928540fa27
pylint: Upgrade to version 2.10.2.
...
This PR sets pylint to version 2.10.2 in all linter actions, and
fixes code in plugins so that this version new checks are either
satisfied or ignored if needed.
2021-09-01 11:27:47 -03:00
Rafael Guterres Jeffman
3beb041ec1
Fix setup.cfg formatting.
2021-05-25 14:13:43 -03:00
Rafael Guterres Jeffman
14c4b60aae
Disable pylint warnings we don't care.
2021-05-25 14:13:43 -03:00
Rafael Guterres Jeffman
3acb9333f4
Disable pylint's c-extension-no-member.
2021-05-25 14:13:42 -03:00
Rafael Guterres Jeffman
afb64419d5
Disable pylint's too-many-lines for modules.
2021-05-25 14:13:42 -03:00
Rafael Guterres Jeffman
b5429618f1
Disable pylint's warnings on import order ang grouping.
2021-05-25 14:13:42 -03:00
Rafael Guterres Jeffman
07abd6c12e
Disable pylint's too-many-arguments.
...
This is a style decision for ansible-freeipa, and in use by most
modules.
2021-05-25 13:55:21 -03:00
Rafael Guterres Jeffman
87504eaa2c
Disable pylint's too-many-statements.
...
This is expected for most modules `main()` function.
2021-05-25 13:55:21 -03:00
Rafael Guterres Jeffman
f1ecc5d986
Disable pylint error no-name-in-module.
...
All instances related to `ansible.module_utils.ansible_freeipa_module`,
which works. Future occurrences, if they happen, will likely not to be
a problem.
2021-05-25 13:55:21 -03:00
Rafael Guterres Jeffman
482bd05b62
Disable pylint's protected-access warning.
...
Protected access is required for AnsibleModule.
2021-05-25 13:55:21 -03:00
Rafael Guterres Jeffman
0dabcd402f
Disable pylint's too-many-locals and too-many-branches.
...
Although both warnings are relevant, the code style choosen for
ansible-freeipa currently require them to be disable.
2021-05-25 13:55:21 -03:00
Rafael Guterres Jeffman
b3a6c9ebe1
Disable pylint broad-except warning.
...
This should be enabled in the future, but currently, nearly all
modules rely on `Exception`, and the changes would be too invasive.
2021-05-25 13:55:21 -03:00
Rafael Guterres Jeffman
b37045bd41
Disable pylint duplicate code verification.
...
Although it is an interesting setup, it currently has too many false
positives, disable comments are not working for duplicate-code, and
there are some expected duplications in the modules.
2021-05-25 13:55:21 -03:00
Rafael Guterres Jeffman
fa9e11363a
Disable pylint warning for wrong import position.
2021-05-25 13:55:21 -03:00
Rafael Guterres Jeffman
efce0bdc05
Disable pylint warnings for missing docstrings.
2021-05-25 13:55:21 -03:00
Rafael Guterres Jeffman
935956b610
Fix pylint's warning invalid-name.
2021-05-25 13:55:21 -03:00
Rafael Guterres Jeffman
b7900f1c64
Add script to run linters.
...
The scprit `lint_check.sh` under utils runs the same linters as the
Github Actions, with the same configuration.
The changes on setup.cfg are required if flake8-docstrings is used,
so its output is the same as pydocstlye.
2020-09-01 17:47:32 -03:00
Sergio Oliveira Campos
5643cfc20d
Adjusted doc strings to follow PEP 257.
2020-04-25 19:07:54 -03:00
Sergio Oliveira Campos
4155f2f3ac
Made code flake8 friendly
2020-04-25 19:07:54 -03:00
Thomas Woerner
645d8e806c
Removed ipa-krb5 from the data_files section
2019-05-03 14:42:50 +02:00
Sean Pryor
3f14b81bc4
Updated setup.cfg to include all the necessary roles
...
and remove unused roles
Change-Id: I68094fc6fcb4292508660bd8ec5d3aeafc85974f
2019-04-18 09:37:30 -04:00
Juan Antonio Osorio Robles
5e48f8e2ba
Introduce setup files that copy roles to relevant directories
...
This enables easy installation of these roles through pip.
2018-02-13 17:15:03 +02:00
