Merge pull request #4 from KB-perByte/add_tox

cleanup dependent files, implement tox

.github/actions/ansible-lint/action.yml

@@ -1,3 +1,4 @@
+---
 name: Ansible lint test job
 description: This ansible-lint workflow
 author: Sagar Paul

.github/actions/release-ah/action.yml

@@ -1,3 +1,4 @@
+---
 name: Release collection Automation Hub
 description: This releases collection on Automation Hub
 author: Sagar Paul

.github/actions/release-galaxy/action.yml

@@ -1,3 +1,4 @@
+---
 name: Release collection Galaxy
 description: This releases collection on Galaxy
 author: Sagar Paul

.github/actions/sanity/action.yml

@@ -1,3 +1,4 @@
+---
 name: Sanity tests
 description: This is sanity testing
 author: Sagar Paul

.gitignore

@@ -20,6 +20,7 @@ parts/
 sdist/
 var/
 wheels/
+pip-wheel-metadata/
 share/python-wheels/
 *.egg-info/
 .installed.cfg
@@ -49,7 +50,6 @@ coverage.xml
 *.py,cover
 .hypothesis/
 .pytest_cache/
-cover/
 
 # Translations
 *.mo
@@ -72,7 +72,6 @@ instance/
 docs/_build/
 
 # PyBuilder
-.pybuilder/
 target/
 
 # Jupyter Notebook
@@ -83,9 +82,7 @@ profile_default/
 ipython_config.py
 
 # pyenv
-#   For a library or package, you might want to ignore these files since the code is
-#   intended to run in multiple environments; otherwise, check them in:
-# .python-version
+.python-version
 
 # pipenv
 #   According to pypa/pipenv#598, it is recommended to include Pipfile.lock in version control.
@@ -94,22 +91,7 @@ ipython_config.py
 #   install all needed dependencies.
 #Pipfile.lock
 
-# poetry
-#   Similar to Pipfile.lock, it is generally recommended to include poetry.lock in version control.
-#   This is especially recommended for binary packages to ensure reproducibility, and is more
-#   commonly ignored for libraries.
-#   https://python-poetry.org/docs/basic-usage/#commit-your-poetrylock-file-to-version-control
-#poetry.lock
-
-# pdm
-#   Similar to Pipfile.lock, it is generally recommended to include pdm.lock in version control.
-#pdm.lock
-#   pdm stores project-wide configurations in .pdm.toml, but it is recommended to not include it
-#   in version control.
-#   https://pdm.fming.dev/#use-with-ide
-.pdm.toml
-
-# PEP 582; used by e.g. github.com/David-OConnor/pyflow and github.com/pdm-project/pdm
+# PEP 582; used by e.g. github.com/David-OConnor/pyflow
 __pypackages__/
 
 # Celery stuff
@@ -145,16 +127,4 @@ dmypy.json
 
 # Pyre type checker
 .pyre/
-
-# pytype static type analyzer
-.pytype/
-
-# Cython debug symbols
-cython_debug/
-
-# PyCharm
-#  JetBrains specific template is maintained in a separate JetBrains.gitignore that can
-#  be found at https://github.com/github/gitignore/blob/main/Global/JetBrains.gitignore
-#  and can be added to the global gitignore or merged into this file.  For a more nuclear
-#  option (not recommended) you can uncomment the following to ignore the entire idea folder.
-#.idea/
+.DS_Store

.pre-commit-config.yaml

@@ -1,91 +1,37 @@
 ---
 repos:
-  - repo: https://github.com/PyCQA/isort
-    rev: 5.12.0
-    hooks:
-      - id: isort
-        args: ["--filter-files"]
-
   - repo: https://github.com/pre-commit/mirrors-prettier
-    rev: "v3.0.0"
+    rev: v3.0.3
     hooks:
       - id: prettier
-        additional_dependencies:
-          - prettier
-          - prettier-plugin-toml
+        always_run: true
 
-  - repo: https://github.com/psf/black
-    rev: 23.7.0
+  - repo: https://github.com/PyCQA/doc8
+    rev: "v1.1.1"
     hooks:
-      - id: black
+      - id: doc8
 
-  - repo: https://github.com/Lucas-C/pre-commit-hooks
-    rev: v1.5.1
+  - repo: https://github.com/codespell-project/codespell
+    rev: v2.2.6
     hooks:
-      - id: remove-tabs
+      - id: codespell
 
-  - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.4.0
+  - repo: https://github.com/pre-commit/pre-commit-hooks.git
+    rev: v4.5.0
     hooks:
+      - id: end-of-file-fixer
       - id: trailing-whitespace
+      - id: mixed-line-ending
+      - id: check-byte-order-marker
+      - id: check-executables-have-shebangs
       - id: check-merge-conflict
-      - id: end-of-file-fixer
-      - id: no-commit-to-branch
-      - id: fix-byte-order-marker
       - id: debug-statements
+        language_version: python3
 
-  - repo: https://github.com/codespell-project/codespell
-    rev: v2.2.5
-    hooks:
-      - id: codespell
-
-  - repo: https://github.com/pycqa/flake8
-    rev: 6.0.0
-    hooks:
-      - id: flake8
-        additional_dependencies:
-          - darglint
-          - flake8-2020 >= 1.6.0
-          - flake8-docstrings # uses pydocstyle
-          - flake8-isort >= 4.1.1
-
-  - repo: https://github.com/asottile/pyupgrade
-    # keep it after flake8
-    rev: v3.9.0
-    hooks:
-      - id: pyupgrade
-        args: ["--py39-plus"]
-
-  - repo: https://github.com/pre-commit/mirrors-mypy
-    rev: v1.4.1
+  - repo: https://github.com/adrienverge/yamllint.git
+    rev: v1.32.0
     hooks:
-      - id: mypy
-        additional_dependencies:
-          - types-PyYAML
-          - pygithub
-          - pytest
-          - types-requests
-
-  - repo: https://github.com/pycqa/pylint
-    rev: v3.0.0a6
-    hooks:
-      - id: pylint
-        additional_dependencies:
-          - PyYAML
-          - pygithub
-          - pytest
-          - semver
-
-  - repo: local
-    hooks:
-      - id: pytest-check
-        name: pytest-check
-        entry: pytest .github scripts -vvvv
-        types: [python]
-        language: python
-        pass_filenames: false
-        always_run: true
-        additional_dependencies:
-          - pytest
-          - pygithub
-          - pyyaml
+      - id: yamllint
+        files: \.(yaml|yml)$
+        types: [file, yaml]
+        entry: yamllint --strict

.yamllint

@@ -0,0 +1,15 @@
+extends: default
+rules:
+  comments:
+    # prettier compatibility
+    min-spaces-from-content: 1
+  # caused false-positives with file sequence items being commented
+  # https://github.com/adrienverge/yamllint/issues/384
+  comments-indentation: disable
+  # ansible standards do require --- prefix
+  document-start: disable
+  # we need to decide if we want to impose one or not
+  line-length: disable
+  truthy:
+    # "on:" is perfectly valid key on github actions:
+    check-keys: false