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

[Scaleway] Fix mismatch default value function namespaces #5462

Conversation

Lunik
Copy link
Contributor

@Lunik Lunik commented Nov 2, 2022

SUMMARY

Fix mismatch default value between documentation and argument_spec

Following #5415 and completing #5446

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

scaleway_function_namespace

ADDITIONAL INFORMATION

CI Fails with the new ansible-core version (2.15.0.dev0) with the following error message :

ERROR: plugins/modules/cloud/scaleway/scaleway_function_namespace.py:0:0: doc-default-does-not-match-spec: Argument 'environment_variables' in argument_spec defines default as ({}) but documentation defines default as (None) (0%)
ERROR: plugins/modules/cloud/scaleway/scaleway_function_namespace.py:0:0: doc-default-does-not-match-spec: Argument 'secret_environment_variables' in argument_spec defines default as ({}) but documentation defines default as (None) (0%)

@ansibullbot
Copy link
Collaborator

@ansibullbot ansibullbot added bug This issue/PR relates to a bug cloud docs_only module module plugins plugin (any type) small_patch Hopefully easy to review labels Nov 2, 2022
@felixfontein
Copy link
Collaborator

I already fixed that directly in main: 2b0bebc

@Lunik
Copy link
Contributor Author

Lunik commented Nov 2, 2022

Didn't see that one. You were definitely quicker 😄

@Lunik Lunik closed this Nov 2, 2022
@felixfontein
Copy link
Collaborator

I noticed it while working on #5461 ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue/PR relates to a bug cloud docs_only module module plugins plugin (any type) small_patch Hopefully easy to review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants