ansible-collections / community.general

Ansible Community General Collection
https://galaxy.ansible.com/ui/repo/published/community/general/
GNU General Public License v3.0
816 stars 1.5k forks source link

use dict comprehension in plugins, part 3 #8833

Closed russoz closed 3 weeks ago

russoz commented 3 weeks ago
SUMMARY

Another PR to mass-replace the construct dict((k, v) for k, v in d if condition) with dict comprehensions: {k: v for k, v in d if condition}

ISSUE TYPE
COMPONENT NAME

plugins/cache/redis.py plugins/lookup/onepassword.py plugins/module_utils/ocapi_utils.py plugins/module_utils/redfish_utils.py plugins/module_utils/scaleway.py plugins/modules/alternatives.py plugins/modules/apache2_mod_proxy.py plugins/modules/consul_acl.py plugins/modules/imc_rest.py plugins/modules/keycloak_user_federation.py plugins/modules/pids.py plugins/modules/pipx.py plugins/modules/pipx_info.py plugins/modules/pkg5_publisher.py plugins/modules/scaleway_compute.py plugins/modules/scaleway_ip.py plugins/modules/scaleway_lb.py plugins/modules/scaleway_security_group.py plugins/modules/scaleway_user_data.py plugins/modules/sensu_silence.py plugins/modules/snmp_facts.py plugins/modules/sorcery.py tests/unit/plugins/module_utils/test_cmd_runner.py tests/unit/plugins/module_utils/test_python_runner.py

ansibullbot commented 3 weeks ago

cc @DenBeke @Ilgmi @abarbare @apollo13 @azenk @bcoca @bhavya06 @dagwieers @dmsimard @eikef @fishman @jasperla @jpdasma @jyundt @laurpaum @mator @mattock @mavit @mraineri @mulby @ndclt @oboukili @ogenstad @rajeevkallur @remyleone @renxulei @samdoran @saranyasridharan @scathatheworm @sgargan @smbambling @thomasbach-dev @tomasg2012 @troy2914 @ujwalkomarla @vaygr @xen0l @xmadsen click here for bot help

patchback[bot] commented 3 weeks ago

Backport to stable-9: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-9/26df6c765767d830ab431570a71859ba82dbfe51/pr-8833

Backported as https://github.com/ansible-collections/community.general/pull/8835

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

felixfontein commented 3 weeks ago

@russoz thanks!