Skip to content

Add missing comma in introduction.rst #2914

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

Merged
merged 1 commit into from
Jul 29, 2025

Conversation

bartolomeomueller
Copy link
Contributor

@bartolomeomueller bartolomeomueller commented Jul 27, 2025

Hey,
I just stumbled upon this sentence in the docu and needed to read it three times to get what it's saying. I think there is a comma missing, but I am not a native speaker, so please double check it.

@ansible-documentation-bot ansible-documentation-bot bot added the new_contributor This PR is the first contribution by a new community member. label Jul 27, 2025
@ansible-documentation-bot
Copy link
Contributor

Thanks for your Ansible docs contribution! We talk about Ansible documentation on Matrix at #docs:ansible.im if you ever want to join us and chat about the docs! We meet on Matrix every Tuesday. See the Ansible calendar for meeting details. We welcome additions to our weekly agenda items too. You can add the dawgs-meeting tag to a forum topic to bring it up at the next meeting.

@ansible-documentation-bot
Copy link
Contributor

Thanks for your contribution, @bartolomeomueller! Please make sure that your pull request includes sufficient and meaningful details in the description.
PR descriptions provide important context and allow other developers and our future selves to understand a change's rationale and what it actually fixes or accomplishes.

Copy link
Collaborator

@felixfontein felixfontein left a comment

Choose a reason for hiding this comment

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

I think this is correct (but then, I'm also not a native English speaker ;) ). It definitely makes the sentence easier to parse though IMO.

@felixfontein felixfontein added backport-2.17 Automatically create a backport for the stable-2.17 branch backport-2.18 Automatically create a backport for the stable-2.18 branch backport-2.19 Automatically create a backport for the stable-2.19 branch labels Jul 28, 2025
@felixfontein felixfontein requested a review from samccann July 28, 2025 20:11
@samccann
Copy link
Contributor

Thanks @bartolomeomueller ! This does make the sentence easier to read.

@samccann samccann merged commit aad8488 into ansible:devel Jul 29, 2025
12 checks passed
Copy link

patchback bot commented Jul 29, 2025

Backport to stable-2.17: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-2.17/aad8488440321ef5722f6a8d2b6a15f7179bf7fb/pr-2914

Backported as #2919

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

Copy link

patchback bot commented Jul 29, 2025

Backport to stable-2.18: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-2.18/aad8488440321ef5722f6a8d2b6a15f7179bf7fb/pr-2914

Backported as #2920

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

Copy link

patchback bot commented Jul 29, 2025

Backport to stable-2.19: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-2.19/aad8488440321ef5722f6a8d2b6a15f7179bf7fb/pr-2914

Backported as #2921

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-2.17 Automatically create a backport for the stable-2.17 branch backport-2.18 Automatically create a backport for the stable-2.18 branch backport-2.19 Automatically create a backport for the stable-2.19 branch new_contributor This PR is the first contribution by a new community member.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants