-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
feat(gandi_livedns): support personal access tokens #8337
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your contribution!
Thanks for the prompt review and spot-on suggestions @felixfontein . Very much appreciated. Should be fixed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me! If nobody objects, I'll merge this in ~a week.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
a409f8f
into
ansible-collections:main
@gilbsgilbs thanks for contributing this! |
…ons#8337) * fix(gandi_livedns): fix unsafe conditionals in tests * feat(gandi_livedns): support personal access tokens Fixes ansible-collections#7639
SUMMARY
ISSUE TYPE
COMPONENT NAME
gandi_livedns
ADDITIONAL INFORMATION
Should be self-explanatory.