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

ansible-test - Improve code formatting #79983

Merged
merged 8 commits into from Feb 13, 2023
Merged

Conversation

mattclay
Copy link
Member

SUMMARY

ansible-test - Improve code formatting.

This applies some, but not all, of the changes suggested by black. It also includes some # fmt: skip comments to protect formatting of key-value pairs when constructing CLI argument lists.

It also includes some manual formatting changes that helps black produce more readable code.

ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME

ansible-test

@ansibot
Copy link
Contributor

ansibot commented Feb 10, 2023

@mattclay This PR was evaluated as a potentially problematic PR for the following reasons:

  • More than 50 changed files.

Such PR can only be merged by human. Contact a Core team member to review this PR on IRC: #ansible-devel on Libera.chat IRC

click here for bot help

@ansibot ansibot added affects_2.15 bug This issue/PR relates to a bug. needs_rebase https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. needs_triage Needs a first human triage before being processed. labels Feb 10, 2023
@mattclay mattclay removed the needs_triage Needs a first human triage before being processed. label Feb 11, 2023
@mattclay mattclay merged commit 715ab99 into ansible:devel Feb 13, 2023
@mattclay mattclay deleted the at-formatting branch February 13, 2023 18:50
@ansible ansible locked and limited conversation to collaborators Feb 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
affects_2.15 bug This issue/PR relates to a bug. needs_rebase https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants