ansible-playbooks/general#85 Linting

defaults/main.yml

@@ -3,8 +3,8 @@ owncloud_ssl: false
 
 owncloud_cron_core:
   active: true
-  month: '*'
-  day: '*'
-  weekday: '*'
-  hour: '*'
-  minute: '0,15,30,45'
+  month: *
+  day: *
+  weekday: *
+  hour: *
+  minute: 0,15,30,45

handlers/main.yml

 ---
 # file: roles/owncloud/handlers/main.yml
 
-- name: 'ownCloud | Restart Apache'
+- name: ownCloud | Restart Apache
   service: name=apache2 state=restarted

tasks/main.yml

 ---
 # file: roles/owncloud/tasks/main.yml
 
-- name: "ownCloud Role"
-  set_fact: role_owncloud_started=true
-  tags: 'always'
+- name: ownCloud Role
+  set_fact:
+    role_owncloud_started: yes
+  tags:
+    - always
 
 - block:
 
-  - name: 'ownCloud | Install Public Repo Key'
+  - name: ownCloud | Install Public Repo Key
     apt_key:
       url='http://download.opensuse.org/repositories/isv:ownCloud:community/xUbuntu_12.04/Release.key'
       state=present
 
-  - name: 'ownCloud | Add ownCloud Repository To Apt'
+  - name: ownCloud | Add ownCloud Repository To Apt
     copy:
       content='deb http://download.opensuse.org/repositories/isv:/ownCloud:/community/xUbuntu_12.04/ /'
       dest='/etc/apt/sources.list.d/owncloud.list'
       mode='644'
 
-  - name: 'ownCloud | Install the package'
+  - name: ownCloud | Install the package
     apt:
       pkg=owncloud
       state=present
       update_cache=yes
 
-  - name: 'ownCloud | Configuration file for apache'
+  - name: ownCloud | Configuration file for apache
     template: src=etc-apache2-sites-available-owncloud
               dest=/etc/apache2/sites-available/owncloud
 
-  - name: 'ownCloud | Enable the ownCloud site for Apache'
+  - name: ownCloud | Enable the ownCloud site for Apache
     command: a2ensite owncloud creates=/etc/apache2/sites-enabled/owncloud
     notify:
-      - 'ownCloud | Restart Apache'
+      - ownCloud | Restart Apache
 
-  - name: 'ownCloud | Install cron job'
+  - name: ownCloud | Install cron job
     cron:
       name='ownCloud cron'
       month='{{ owncloud_cron_core.month|default(omit) }}'
@@ -45,6 +47,7 @@
       user='www-data'
       disabled='{{ crons_disabled|default(false) }}'
     when: owncloud_cron_core.active
-    tags: 'cron'
+    tags:
+      - cron
 
-  when: '"owncloud" not in excluded_roles'
+  when: not excluded_roles or "owncloud" not in excluded_roles