-
Notifications
You must be signed in to change notification settings - Fork 65
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
Add docs for vault_login_token
filter
#263
Add docs for vault_login_token
filter
#263
Conversation
Docs Build 📝Thank you for contribution!✨ This PR has been merged and the docs are now incorporated into |
This comment was marked as outdated.
This comment was marked as outdated.
The changes needed in core are in
|
6d2a897
to
f613895
Compare
Codecov Report
@@ Coverage Diff @@
## main #263 +/- ##
=======================================
Coverage 98.66% 98.66%
=======================================
Files 73 73
Lines 3679 3679
Branches 313 313
=======================================
Hits 3630 3630
Misses 40 40
Partials 9 9
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
I'm going to hold off on this for now, possibly until version 4.0.0 / ansible-core 2.14, since the feature will not be available until then. I could add just the sidecar docs for now without linking to them with references (because that breaks if docs are rendered with an ansible version earlier than devel, since the target ref won't exist). But if I do that, I am not testing them in my collection. To remedy that, I could choose to use If the |
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.
This is great information to have in the docs, and great functionality to have in Ansible! Thanks @briantist.
I've made a few wording suggestions. My interpretation of what the filter does may not be correct, but I hope the suggestions will help clarify the docs a bit.
@acozine thanks I will have to look over your suggestions in more detail later, but I think what's there will make more sense if you look at the existing filter guide (which is where I pulled these description) where I've gone into more detail: https://ansible-collections.github.io/community.hashi_vault/pr/263/collections/community/hashi_vault/docsite/filter_guide.html The challenge is writing something for a documentation page where I can't include all of that context. |
f613895
to
3bb1f65
Compare
Current failure is not related to this PR as far as I can tell: |
not very familiar with the doc generation machinery, but as far as I can tell, LGTM |
9d4b737
to
8b8b54b
Compare
@felixfontein @russoz thanks for reviewing this! |
SUMMARY
Resolves #260
Also updates the committed docs build stuff (the output ofthis was moved to #306antsibull-docs sphinx-init
).ISSUE TYPE
COMPONENT NAME
vault_login_token