add testing and improve linting

.github/FUNDING.yml

@@ -1,5 +1,5 @@
+---
 # These are supported funding model platforms
 
 github: [do1jlr]
 liberapay: L3D
-custom: https://paypal.me/c3woc

.github/workflows/ansible-alpine-latest.yml

@@ -0,0 +1,18 @@
+---
+name: Ansible check alpine:latest
+
+# yamllint disable-line rule:truthy
+on: [push, pull_request]
+
+jobs:
+  build:
+
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v2
+
+      - name: ansible check with alpine:latest
+        uses: roles-ansible/check-ansible-alpine-latest-action@master
+        with:
+          targets: "./"

.github/workflows/ansible-archlinux-latest.yml

@@ -0,0 +1,18 @@
+---
+name: Ansible check archlinux:latest
+
+# yamllint disable-line rule:truthy
+on: [push, pull_request]
+
+jobs:
+  build:
+
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v2
+
+      - name: ansible check with archlinux:latest
+        uses: roles-ansible/check-ansible-archlinux-latest-action@master
+        with:
+          targets: "./"

.github/workflows/ansible-centos-centos7.yml

@@ -0,0 +1,18 @@
+---
+name: Ansible check centos:centos7
+
+# yamllint disable-line rule:truthy
+on: [push, pull_request]
+
+jobs:
+  build:
+
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v2
+
+      - name: ansible check with ubuntu:centos7
+        uses: roles-ansible/check-ansible-centos-centos7-action@master
+        with:
+          targets: "./"

.github/workflows/ansible-centos-centos8.yml

@@ -0,0 +1,18 @@
+---
+name: Ansible check centos:centos8
+
+# yamllint disable-line rule:truthy
+on: [push, pull_request]
+
+jobs:
+  build:
+
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v2
+
+      - name: ansible check with ubuntu:centos8
+        uses: roles-ansible/check-ansible-centos-centos8-action@master
+        with:
+          targets: "./"

.github/workflows/ansible-centos-latest.yml

@@ -0,0 +1,18 @@
+---
+name: Ansible check centos:latest
+
+# yamllint disable-line rule:truthy
+on: [push, pull_request]
+
+jobs:
+  build:
+
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v2
+
+      - name: ansible check with ubuntu:latest
+        uses: roles-ansible/check-ansible-centos-latest-action@master
+        with:
+          targets: "./"

.github/workflows/ansible-debian-buster.yml

@@ -0,0 +1,18 @@
+---
+name: Ansible check debian:buster
+
+# yamllint disable-line rule:truthy
+on: [push, pull_request]
+
+jobs:
+  build:
+
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v2
+
+      - name: ansible check with debian:buster
+        uses: roles-ansible/check-ansible-debian-buster-action@master
+        with:
+          targets: "./"

.github/workflows/ansible-debian-latest.yml

@@ -0,0 +1,18 @@
+---
+name: Ansible check debian:latest
+
+# yamllint disable-line rule:truthy
+on: [push, pull_request]
+
+jobs:
+  build:
+
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v2
+
+      - name: ansible check with debian:latest
+        uses: roles-ansible/check-ansible-debian-latest-action@master
+        with:
+          targets: "./"

.github/workflows/ansible-debian-sid.yml

@@ -0,0 +1,18 @@
+---
+name: Ansible check debian:sid
+
+# yamllint disable-line rule:truthy
+on: [push, pull_request]
+
+jobs:
+  build:
+
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v2
+
+      - name: ansible check with debian:sid
+        uses: roles-ansible/check-ansible-debian-sid-action@master
+        with:
+          targets: "./"

.github/workflows/ansible-debian-stable.yml

@@ -0,0 +1,24 @@
+---
+name: Ansible check debian:stable
+
+# yamllint disable-line rule:truthy
+on:
+  push:
+    branches: '*'
+  pull_request:
+    branches: '*'
+  schedule:
+    - cron: '23 6 * */1 *'
+
+jobs:
+  build:
+
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v2
+
+      - name: ansible check with debian:stable
+        uses: roles-ansible/check-ansible-debian-stable-action@master
+        with:
+          targets: "./"

.github/workflows/ansible-debian-stretch.yml

@@ -0,0 +1,18 @@
+---
+name: Ansible check debian:stretch
+
+# yamllint disable-line rule:truthy
+on: [push, pull_request]
+
+jobs:
+  build:
+
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v2
+
+      - name: ansible check with debian:stretch
+        uses: roles-ansible/check-ansible-debian-stretch-action@master
+        with:
+          targets: "./"

.github/workflows/ansible-fedora-31.yml

@@ -0,0 +1,18 @@
+---
+name: Ansible check fedora:31
+
+# yamllint disable-line rule:truthy
+on: [push, pull_request]
+
+jobs:
+  build:
+
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v2
+
+      - name: ansible check with fedora:31
+        uses: roles-ansible/check-ansible-fedora-31-action@master
+        with:
+          targets: "./"

.github/workflows/ansible-fedora-32.yml

@@ -0,0 +1,18 @@
+---
+name: Ansible check fedora:32
+
+# yamllint disable-line rule:truthy
+on: [push, pull_request]
+
+jobs:
+  build:
+
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v2
+
+      - name: ansible check with fedora:32
+        uses: roles-ansible/check-ansible-fedora-32-action@master
+        with:
+          targets: "./"

.github/workflows/ansible-fedora-33.yml

@@ -0,0 +1,18 @@
+---
+name: Ansible check fedora:33
+
+# yamllint disable-line rule:truthy
+on: [push, pull_request]
+
+jobs:
+  build:
+
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v2
+
+      - name: ansible check with fedora:33
+        uses: roles-ansible/check-ansible-fedora-33-action@master
+        with:
+          targets: "./"

.github/workflows/ansible-fedora-latest.yml

@@ -0,0 +1,18 @@
+---
+name: Ansible check fedora:latest
+
+# yamllint disable-line rule:truthy
+on: [push, pull_request]
+
+jobs:
+  build:
+
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v2
+
+      - name: ansible check with fedora:latest
+        uses: roles-ansible/check-ansible-fedora-latest-action@master
+        with:
+          targets: "./"

.github/workflows/ansible-linting-check.yml

@@ -0,0 +1,28 @@
+---
+name: Ansible Lint check
+
+# yamllint disable-line rule:truthy
+on:
+  push:
+    branches: '*'
+  pull_request:
+    branches: '*'
+  schedule:
+    - cron: '42 6 * */1 *'
+
+jobs:
+  build:
+
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v2
+
+      - name: Lint Ansible Playbook
+        uses: ansible/ansible-lint-action@master
+        with:
+          targets: "."
+          # [required]
+          # Paths to ansible files (i.e., playbooks, tasks, handlers etc..)
+          args: ""
+          # [optional]

.github/workflows/ansible-ubuntu-bionic.yml

@@ -0,0 +1,18 @@
+---
+name: Ansible check ubuntu:bionic
+
+# yamllint disable-line rule:truthy
+on: [push, pull_request]
+
+jobs:
+  build:
+
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v2
+
+      - name: ansible check with ubuntu:bionic
+        uses: roles-ansible/check-ansible-ubuntu-bionic-action@master
+        with:
+          targets: "./"

.github/workflows/ansible-ubuntu-latest.yml

@@ -0,0 +1,18 @@
+---
+name: Ansible check ubuntu:latest
+
+# yamllint disable-line rule:truthy
+on: [push, pull_request]
+
+jobs:
+  build:
+
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v2
+
+      - name: ansible check with ubuntu:latest
+        uses: roles-ansible/check-ansible-ubuntu-latest-action@master
+        with:
+          targets: "./"

.github/workflows/ansible-ubuntu-trusty.yml

@@ -0,0 +1,18 @@
+---
+name: Ansible check ubuntu:trusty
+
+# yamllint disable-line rule:truthy
+on: [push, pull_request]
+
+jobs:
+  build:
+
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v2
+
+      - name: ansible check with ubuntu:trusty
+        uses: roles-ansible/check-ansible-ubuntu-trusty-action@master
+        with:
+          targets: "./"

.github/workflows/galaxy.yml

@@ -0,0 +1,20 @@
+---
+name: Galaxy release
+
+# yamllint disable-line rule:truthy
+on:
+  push:
+    branches: ['main']
+  release:
+    types: ['created']
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+    steps:
+      - name: checkout
+        uses: actions/checkout@v2
+      - name: galaxy
+        uses: robertdebock/galaxy-action@1.1.0
+        with:
+          galaxy_api_key: ${{ secrets.galaxy_api_key }}

.github/workflows/yamllint.yaml

@@ -0,0 +1,24 @@
+---
+name: 'Yamllint GitHub Actions'
+
+# yamllint disable-line rule:truthy
+on:
+  push:
+    branches: '*'
+  pull_request:
+    branches: '*'
+  schedule:
+    - cron: '23 6 * */1 *'
+
+jobs:
+  yamllint:
+    name: 'Yamllint'
+    runs-on: ubuntu-latest
+    steps:
+      - name: 'Checkout'
+        uses: actions/checkout@master
+      - name: 'Yamllint'
+        uses: karancode/yamllint-github-action@master
+        with:
+          yamllint_file_or_dir: '.'
+          yamllint_config_filepath: './.yamllint'

.yamllint

@@ -0,0 +1,8 @@
+---
+extends: default
+
+rules:
+  # 150 chars should be enough, but don't fail if a line is longer
+  line-length:
+    max: 150
+    level: warning

meta/main.yml

@@ -1,3 +1,4 @@
+---
 galaxy_info:
   role_name: ntp
   author: do1jlr

tasks/packages/setup-FreeBSD.yml

@@ -2,14 +2,14 @@
 # Tasks file for install ntp daemon on FreeBSD system
 
 - name: Install ntp daemon on FreeBSD
-  become: yes
-  package:
+  become: true
+  ansible.builtin.package:
     name: "{{ ntp_package }}"
     state: present
 
 - name: Service ntp on FreeBSD
-  become: yes
-  service:
+  become: true
+  ansible.builtin.service:
     name: "{{ ntp_service }}"
-    enabled: yes
+    enabled: true
     state: started

vars/Archlinux.yml

@@ -1,3 +1,4 @@
+---
 ntp_package: ntp
 ntp_service: ntpd
 

vars/Darwin.yml

@@ -1,3 +1,4 @@
+---
 ntp_package:
 ntp_service: ntp
 

vars/Debian.yml

@@ -1,3 +1,4 @@
+---
 ntp_package: ntp
 ntp_service: ntp
 

vars/FreeBSD.yml

@@ -1,3 +1,4 @@
+---
 ntp_package: ntp
 ntp_service: ntpd
 

vars/Gentoo.yml

@@ -1,3 +1,4 @@
+---
 ntp_package: net-misc/ntp
 ntp_service: ntp-client
 

vars/OpenBSD.yml

@@ -1,3 +1,4 @@
+---
 ntp_package: ntp
 ntp_service: ntpd
 

vars/RedHat.yml

@@ -1,3 +1,4 @@
+---
 ntp_package: ntp
 ntp_service: ntpd
 

vars/Suse.yml

@@ -1,3 +1,4 @@
+---
 ntp_package: ntp
 ntp_service: ntpd
 

vars/fallback.yml

@@ -1,3 +1,4 @@
+---
 ntp_package: ntp
 ntp_service: ntp
 

vars/main.yml

@@ -1,5 +1,5 @@
 ---
-playbook_version_number: 2050  # should be int
+playbook_version_number: 2051  # should be int
 playbook_version_path: 'role-ntp_chaos-bodensee_github.com.version'
 
 ntp__vars: