1
0
Fork 0
mirror of https://github.com/ansible-collections/community.general.git synced 2024-09-14 20:13:21 +02:00
community.general/docs/proposals/rename_always_run.md
René Moser 130c995208 proposal: ignore_checkmode is a better name
as suggested by @bcoca
2016-03-05 09:23:54 +01:00

753 B

Rename always_run to ignore_checkmode

Author: René Moser <@resmo>

Date: 02/03/2016

Motivation

The task argument always_run is misleading.

Ansible is known to be readable by users without deep knowledge of creating playbooks, they do not understand what always_run does at the first glance.

Problems

The following looks scary if you have no idea, what always_run does:

- shell: dangerous_cleanup.sh
  when: cleanup == "yes"
  always_run: yes

You have a conditional but also a word that says always. This is a conflict in terms of understanding.

Solution Proposal

Deprecate always_run by rename it to ignore_checkmode:

- shell: dangerous_cleanup.sh
  when: cleanup == "yes"
  ignore_checkmode: yes