Move scripts, remove unused git files, and add a pre-commit hook (#52)

* Remove requirements.txt - never been used
* Move get_version.py to scripts folder
* Move assert.sh into scripts folder
* Remove dependabot config
* Add scripts folder readme
* Remove .gitattributes
* Remove .gitignore
* Add pre-commit hooks for workflows and actions

.github/workflows/tag_release.yml

@@ -12,8 +12,8 @@ jobs:
       - uses: actions/checkout@v2
       - name: Update tag
         run: |
-          major_tag="$(python get_version.py "${GITHUB_REF}" major)"
-          minor_tag="$(python get_version.py "${GITHUB_REF}" minor)"
+          major_tag="$(python .github/scripts/get_version.py "${GITHUB_REF}" major)"
+          minor_tag="$(python .github/scripts/get_version.py "${GITHUB_REF}" minor)"
           git tag $major_tag
           git tag $minor_tag
           git push origin HEAD:refs/heads/master --tags --force

.github/workflows/test.yml

@@ -14,7 +14,7 @@ env:
 
   # Normal variables (used in steps/*)
   LATEST_PYTHON: "3.9"
-  LATEST_POETRY: "$(cat requirements.txt | sed 's/poetry==//g')"
+  LATEST_POETRY: "1.1.8"
 
 jobs:
   # The set-env job translates our json variables to a format we
@@ -94,7 +94,7 @@ jobs:
         run: |
           output=$(poetry config virtualenvs.create)
           echo $output
-          source "./assert.sh"
+          source "./.github/scripts/assert.sh"
           assert_in "false" "$output"
 
   # Makes sure creating a venv in-project works
@@ -120,7 +120,7 @@ jobs:
           python-version: ${{ env.LATEST_PYTHON }}
       - run: |
           output="$(poetry install)"
-          source "./assert.sh"
+          source "./.github/scripts/assert.sh"
           echo "$output"
           assert_in "/install-poetry/install-poetry/.venv" "$output"
           source $VENV
@@ -139,7 +139,7 @@ jobs:
       # If you're submitting a PR and this fails because Poetry release a new version
       # feel free to update it
       - run: |
-          source "./assert.sh"
+          source "./.github/scripts/assert.sh"
           assert_in "1.1.8" "$(poetry --version)"
 
   # Make sure scripts are not deleted.
@@ -184,5 +184,5 @@ jobs:
         run: |
           output=$(poetry config installer.parallel)
           echo $output
-          source "./assert.sh"
+          source "./.github/scripts/assert.sh"
           assert_in "false" "$output"

.pre-commit-config.yaml

@@ -9,3 +9,8 @@ repos:
       - id: trailing-whitespace
       - id: mixed-line-ending
       - id: trailing-whitespace
+  - repo: https://github.com/sirosen/check-jsonschema
+    rev: '0.4.1'
+    hooks:
+      - id: check-github-actions
+      - id: check-github-workflows

scripts/README.md

@@ -0,0 +1,4 @@
+The scripts in the scripts folder should not ever change.
+
+Previous versions of the action would curl scripts from the repos main branch,
+meaning that if we remove them, old pinned versions of the action will break.