ansibot comments

Results 1037 comments of


                                            ansibot

Fix PyYAML pure Python parser compatibility with Origin validation

The test `ansible-test sanity --test pep8` [[explain](https://docs.ansible.com/ansible-core/devel/dev_guide/testing/sanity/pep8.html)] failed with 42 errors: ``` test/units/plugins/filter/test_core.py:91:1: E302: expected 2 blank lines, found 1 test/units/plugins/filter/test_core.py:91:42: W291: trailing whitespace test/units/plugins/filter/test_core.py:92:16: W291: trailing whitespace test/units/plugins/filter/test_core.py:93:24: W291:...

Not able to use "%:z" in strftime in ansible

Files identified in the description: None If these files are incorrect, please update the `component name` section of the description or use the [component](https://github.com/ansible/ansibotmini#commands) bot command.

deb822_repository replaces valid characters in sources list filename

Files identified in the description: * [`lib/ansible/modules/deb822_repository.py`](https://github.com/ansible/ansible/blob/devel/lib/ansible/modules/deb822_repository.py) If these files are incorrect, please update the `component name` section of the description or use the [component](https://github.com/ansible/ansibotmini#commands) bot command.

Potential improvements for changelog/website situation during RC -> final transition

Files identified in the description: * [`lib/ansible/modules/unarchive.py`](https://github.com/ansible/ansible/blob/devel/lib/ansible/modules/unarchive.py) * [`lib/ansible/plugins/action/unarchive.py`](https://github.com/ansible/ansible/blob/devel/lib/ansible/plugins/action/unarchive.py) If these files are incorrect, please update the `component name` section of the description or use the [component](https://github.com/ansible/ansibotmini#commands) bot command.

Fix user modification on Alpine

The test `ansible-test sanity --test pep8` [[explain](https://docs.ansible.com/ansible-core/devel/dev_guide/testing/sanity/pep8.html)] failed with 4 errors: ``` lib/ansible/modules/user.py:3298:15: E111: indentation is not a multiple of 4 lib/ansible/modules/user.py:3299:15: E111: indentation is not a multiple of 4...

Enable group-level batching in playbook execution

Files identified in the description: * [`lib/ansible/cli/playbook.py`](https://github.com/ansible/ansible/blob/devel/lib/ansible/cli/playbook.py) If these files are incorrect, please update the `component name` section of the description or use the [component](https://github.com/ansible/ansibotmini#commands) bot command.

Update validation logic for parameter checks to allow for None value for `required_if`

The test `ansible-test sanity --test pep8` [[explain](https://docs.ansible.com/ansible-core/devel/dev_guide/testing/sanity/pep8.html)] failed with 1 error: ``` lib/ansible/module_utils/common/validation.py:275:67: E711: comparison to None should be 'if cond is None:' ``` The test `ansible-test sanity --test pylint`...