Merge pull request #55 from MonolithProjects/develop

Added variables validation
MonolithProjects · Dec 24, 2020 · 5a92df5 · 5a92df5
2 parents 5779228 + 93e1758
commit 5a92df5
Show file tree

Hide file tree

Showing 4 changed files with 18 additions and 4 deletions.
diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
@@ -1,9 +1,6 @@
 name: molecule test
 
 on:
-    push:
-        branches:
-            - master
     pull_request:
         branches:
             - master

diff --git a/tasks/assert.yml b/tasks/assert.yml
@@ -0,0 +1,12 @@
+---
+- name: Check variables
+  assert:
+    that:
+      - github_account is defined
+      - access_token is defined
+      - access_token | length > 0
+    quiet: true
+  run_once: yes
+  tags:
+    - install
+    - uninstall
diff --git a/tasks/collect_info_org.yml b/tasks/collect_info_org.yml
@@ -37,7 +37,7 @@
 
 - name: Build service name
   set_fact:
-    runner_service: "actions.runner.{{ github_owner | default(github_account[:45]) }}.{{ runner_name }}.service"
+    runner_service: "actions.runner.{{ (github_owner | default(github_account))[:45] }}.{{ runner_name }}.service"
   tags:
     - install
     - uninstall
diff --git a/tasks/main.yml b/tasks/main.yml
@@ -1,4 +1,9 @@
 ---
+- include_tasks: assert.yml
+  tags:
+    - install
+    - uninstall
+
 - include_tasks: collect_info_org.yml
   when: runner_org
   tags: