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

Add split_pem filter #549

Merged
merged 4 commits into from
Dec 27, 2022
Merged

Conversation

felixfontein
Copy link
Contributor

SUMMARY

Implements parts of #542.

ISSUE TYPE
  • New Plugin Pull Request
COMPONENT NAME

split_pem

@github-actions
Copy link

github-actions bot commented Dec 27, 2022

Docs Build 📝

Thank you for contribution!✨

This PR has been merged and the docs are now incorporated into main:
https://ansible-collections.github.io/community.crypto/branch/main

Comment on lines +6 to +9
####################################################################
# WARNING: These are designed specifically for Ansible tests #
# and should not be used as examples of how to write Ansible roles #
####################################################################
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

total nit and I don't care if it stays, but is this warning still useful? I don't really see it in anything new

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know how useful this really is, but I think it's better to keep it :)

tests/integration/targets/filter_split_pem/tasks/main.yml Outdated Show resolved Hide resolved
@felixfontein felixfontein merged commit 7cc9a70 into ansible-collections:main Dec 27, 2022
@felixfontein felixfontein deleted the filter branch December 27, 2022 20:57
@felixfontein
Copy link
Contributor Author

@briantist thanks for reviewing this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants