Skip to content

Commit

Permalink
Merge pull request #278 from cisagov/improvement/ansible_boolean_pref…
Browse files Browse the repository at this point in the history
…er_true_false

Prefer `true`/`false` to `yes`/`no` in YAML files
  • Loading branch information
mcdonnnj authored Oct 27, 2023
2 parents 93bfe7a + 4ada601 commit 74a2d88
Show file tree
Hide file tree
Showing 5 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion src/aws.yml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
- hosts: all
name: AWS-specific roles
become: yes
become: true
become_method: ansible.builtin.sudo
tasks:
- name: Install Amazon EFS utilities
Expand Down
2 changes: 1 addition & 1 deletion src/base.yml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
- hosts: all
name: Setup base image
become: yes
become: true
become_method: ansible.builtin.sudo
tasks:
- name: Install and configure automated security updates
Expand Down
2 changes: 1 addition & 1 deletion src/example.yml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
- hosts: all
name: Project-specific roles
become: yes
become: true
become_method: ansible.builtin.sudo
tasks:
# The cisagov/ansible-role-example Ansible role is just a no-op,
Expand Down
2 changes: 1 addition & 1 deletion src/python.yml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
- hosts: all
name: Install pip3/python3 and remove pip2/python2
become: yes
become: true
become_method: ansible.builtin.sudo
tasks:
# If pip were to be installed first, then the OS _could_ pull
Expand Down
2 changes: 1 addition & 1 deletion src/upgrade.yml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
- hosts: all
name: Upgrade base image
become: yes
become: true
become_method: ansible.builtin.sudo
tasks:
- name: Upgrade all packages
Expand Down

0 comments on commit 74a2d88

Please sign in to comment.