Merge pull request #495 from ripleymj/fix-linter-warnings

Fix linter warnings

Author: laurelmay

roles/adv_prog_pkgs/meta/main.yml

@@ -4,8 +4,7 @@ galaxy_info:
   description: Installs the advanced programming packages mostly used in CS261
   company: James Madison University
   license: MIT
-  min_ansible_version: 2.1
-  github_branch: main
+  min_ansible_version: '2.9'
   platforms:
     - name: Ubuntu
       versions:

roles/basic_prog_pkgs/meta/main.yml

@@ -4,8 +4,7 @@ galaxy_info:
   description: Installs introductory programming packages
   company: James Madison University
   license: MIT
-  min_ansible_version: 2.1
-  github_branch: main
+  min_ansible_version: '2.9'
   platforms:
     - name: Ubuntu
       versions:

roles/common/meta/main.yml

@@ -4,8 +4,7 @@ galaxy_info:
   description: Installs the shortcuts to easily re-run the VM tasks
   company: James Madison University
   license: MIT
-  min_ansible_version: 2.1
-  github_branch: main
+  min_ansible_version: '2.9'
   platforms:
     - name: Ubuntu
       versions:

roles/common/templates/CustomMintReportInfo.py

@@ -29,7 +29,7 @@
 
 # Since we don't have much choice in the structure of this class since we're
 # specializing a required class, we need to ignore related pylint errors.
-# pylint: disable=no-self-use,unused-argument
+# pylint: disable=unused-argument
 class Report(InfoReport):
     """
     Override of mintreport's InfoReport

roles/eclipse/meta/main.yml

@@ -4,8 +4,7 @@ galaxy_info:
   description: Installs the Eclipse IDE
   company: James Madison University
   license: MIT
-  min_ansible_version: 2.1
-  github_branch: main
+  min_ansible_version: '2.9'
   platforms:
     - name: Ubuntu
       versions:

roles/eclipse/tasks/main.yml

@@ -8,7 +8,8 @@
   ansible.builtin.stat:
     path: '{{ eclipse.zip }}'
   register: st
-- block:
+- name: Download and unpack Eclipse
+  block:
     - name: Fetch Eclipse bundle
       ansible.builtin.get_url:
         url: '{{ eclipse.url }}'

roles/filezilla/meta/main.yml

@@ -4,8 +4,7 @@ galaxy_info:
   description: Installs Filezilla
   company: James Madison University
   license: MIT
-  min_ansible_version: 2.1
-  github_branch: main
+  min_ansible_version: '2.9'
   platforms:
     - name: Ubuntu
       versions:

roles/finch/meta/main.yml

@@ -4,8 +4,7 @@ galaxy_info:
   description: Installs the tools and scripts needed to use Finch robots
   company: James Madison University
   license: MIT
-  min_ansible_version: 2.1
-  github_branch: main
+  min_ansible_version: '2.9'
   platforms:
     - name: Ubuntu
       versions:

roles/jgrasp/meta/main.yml

@@ -4,8 +4,7 @@ galaxy_info:
   description: Installs the jGRASP IDE
   company: James Madison University
   license: MIT
-  min_ansible_version: 2.1
-  github_branch: main
+  min_ansible_version: '2.9'
   platforms:
     - name: Ubuntu
       versions:

roles/jgrasp/tasks/main.yml

@@ -12,7 +12,8 @@
   ansible.builtin.stat:
     path: '{{ jgrasp.install_path }}/internal_bin/sys_run'
   register: wedge
-- block:
+- name: Download and unpack jGRASP
+  block:
     - name: Fetch jGRASP zip
       ansible.builtin.get_url:
         url: '{{ jgrasp.url }}'
@@ -32,7 +33,8 @@
         group: root
         mode: "0755"
   when: zip.stat.checksum|default("") != jgrasp.hash
-- block:
+- name: Configure and build jGRASP wedge
+  block:
     - name: Configure jGRASP wedge build
       command:
         cmd: './configure'