Skip to content

Propose fix a couple of typos#16439

Open
jeis4wpi wants to merge 8 commits into
ansible:develfrom
jeis4wpi:devel
Open

Propose fix a couple of typos#16439
jeis4wpi wants to merge 8 commits into
ansible:develfrom
jeis4wpi:devel

Conversation

@jeis4wpi

@jeis4wpi jeis4wpi commented Apr 29, 2026

Copy link
Copy Markdown
SUMMARY

Propose fix a couple of typos

ISSUE TYPE
  • Docs Fix or other nominal change
COMPONENT NAME
  • Docs
  • Other
STEPS TO REPRODUCE AND EXTRA INFO

Summary by CodeRabbit

  • Documentation

    • Corrected multiple spelling errors in module examples and documentation strings, including credential names and workflow-related descriptions.
  • Chores

    • Fixed typos in test task display names for improved consistency.

Signed-off-by: John E <jeis4wpi@outlook.com>
@github-actions github-actions Bot added component:awx_collection issues related to the collection for controlling AWX community labels Apr 29, 2026
@coderabbitai

coderabbitai Bot commented Apr 29, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Enterprise

Run ID: 4291f48f-42bb-46ae-9a1a-33c3cf673212

📥 Commits

Reviewing files that changed from the base of the PR and between eac8968 and fb2f3e7.

📒 Files selected for processing (9)
  • awx_collection/plugins/modules/bulk_job_launch.py
  • awx_collection/plugins/modules/job_launch.py
  • awx_collection/plugins/modules/workflow_job_template.py
  • awx_collection/plugins/modules/workflow_job_template_node.py
  • awx_collection/tests/integration/targets/demo_data/tasks/main.yml
  • awx_collection/tests/integration/targets/role/tasks/main.yml
  • awx_collection/tests/integration/targets/role_team_assignment/tasks/main.yml
  • awx_collection/tests/integration/targets/role_user_assignment/tasks/main.yml
  • awx_collection/tests/integration/targets/team/tasks/main.yml

📝 Walkthrough

Walkthrough

This pull request corrects spelling errors across module documentation, examples, and integration test task names. All changes are purely textual corrections to documentation strings, example data, and task display names with no modifications to functional logic or control flow.

Changes

Cohort / File(s) Summary
Module Documentation & Examples
awx_collection/plugins/modules/bulk_job_launch.py, awx_collection/plugins/modules/job_launch.py, awx_collection/plugins/modules/workflow_job_template.py, awx_collection/plugins/modules/workflow_job_template_node.py
Spelling corrections in documentation and example strings: "suplementary cred" → "supplementary cred", "coresponding" → "corresponding", "job tempalte" → "job template", and typo fix in skip_tags documentation.
Integration Test Task Names
awx_collection/tests/integration/targets/demo_data/tasks/main.yml, awx_collection/tests/integration/targets/role/tasks/main.yml, awx_collection/tests/integration/targets/role_team_assignment/tasks/main.yml, awx_collection/tests/integration/targets/role_user_assignment/tasks/main.yml, awx_collection/tests/integration/targets/team/tasks/main.yml
Spelling corrections in Ansible task name fields: "deletd" → "deleted", "non existant" → "non existent", "Assigment" → "Assignment", "non-existant" → "non-existent", "reesult" → "result".

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: fixing multiple typos across documentation and test files throughout the codebase.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share
Review rate limit: 7/8 reviews remaining, refill in 7 minutes and 30 seconds.

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community component:awx_collection issues related to the collection for controlling AWX

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant