Skip to content
This repository has been archived by the owner on Feb 16, 2023. It is now read-only.

Update documentation for grammar and additional clarity #1536

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

davemachado
Copy link

Small tweaks to be consistent in oxford comma usage already at work in
the docs. More importantly, adding some clarification here and there
to try and make things even more dead simple to read :)

Thanks for the review and for maintaining such a useful project! I hope I'll have the opportunity to contribute again soon.

Small tweaks to be consistent in oxford comma usage already at work in
the docs. More importantly, adding some clarification here and there
to try and make things even more dead simple to read :)
@davemachado
Copy link
Author

Looks like the CI run Ansible Role / test (pull_request) is failing for unrelated reasons:

ansible_compat.errors.InvalidPrerequisiteError: Computed fully qualified role name of C0nsultant.ansible does not follow current galaxy requirements.
Please edit meta/main.yml and assure we can correctly determine full role name:

Every other job passed though, so I think we're in good shape.

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

Successfully merging this pull request may close these issues.

1 participant