Improve the functionality and maintainability of the codebase #19
Conversation
Sylfwood
force-pushed
the
devops/features
branch
from
7ca7ade to
feda502
Compare
DannyBedard
left a comment
DannyBedard
left a comment
There was a problem hiding this comment.
Manque juste d'implémenter la version (1.2.2?) dans galaxy.yml
| EXAMPLES = r""" | ||
| - name: Fetch dvls server information | ||
| server: | ||
| devolutions.dvls.fetch_server: |
There was a problem hiding this comment.
Je ne crois pas que ça marcherait, si c'est le cas ce serait breaking
There was a problem hiding this comment.
C'est le sanity check qui m'a montré ca! C'est pour que l'exemple de la doc suive comme du monde le code. Si je me fais au playbouk, c'est de même qu'on le fait. Donc l'exemple est bonne.
|
|
||
| def run_module(): | ||
| module_args = dict( | ||
| argument_spec = dict( |
There was a problem hiding this comment.
Je le changerait sur les autres aussi rendu là
| module = AnsibleModule(argument_spec=module_args, supports_check_mode=True) | ||
| module = AnsibleModule(argument_spec=argument_spec, supports_check_mode=True) | ||
|
|
||
| if not HAS_REQUESTS_LIBRARY: |
There was a problem hiding this comment.
Est-ce qu'on peut ajouter cette condition dans les autres modules aussi?
There was a problem hiding this comment.
C'est seulement pertinent pour les libs externes, dans ce cas-ci, celle qui font références à requests. Les autres modules ne l'import pas, donc pas besoin.
Sylfwood
force-pushed
the
devops/features
branch
from
4c401e1 to
975485f
Compare
This pull request includes several changes to improve the functionality and maintainability of the codebase, particularly in the
.github/workflows/test.ymlfile and various module utility files. The changes include updates to the workflow, error handling improvements, and code formatting enhancements.