Pre-commit as GH Action to test

.github/workflows/pre-commit.yml

@@ -0,0 +1,14 @@
+name: pre-commit
+
+on:
+  pull_request:
+  push:
+    branches: [main]
+
+jobs:
+  pre-commit:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v3
+    - uses: actions/setup-python@v3
+    - uses: pre-commit/action@v3.0.0

.pre-commit-config.yaml

@@ -7,18 +7,22 @@ repos:
       - id: check-case-conflict
       - id: check-json
       - id: check-yaml
+        exclude: ^.cicd/
       - id: check-toml
       - id: end-of-file-fixer
       - id: trailing-whitespace
       - id: mixed-line-ending
       - id: detect-aws-credentials
+        args:
+          - "--allow-missing-credentials"
       - id: detect-private-key
       - id: fix-byte-order-marker
 
   - repo: https://github.com/asottile/pyupgrade
     rev: v3.9.0
     hooks:
       - id: pyupgrade
+        exclude: ^tests/
         args: [ "--py38-plus", "--keep-runtime-typing" ]
 
   - repo: https://github.com/psf/black
@@ -32,10 +36,6 @@ repos:
       - id: blacken-docs
         additional_dependencies: [ black ]
 
-  - repo: https://github.com/asottile/seed-isort-config
-    rev: v2.2.0
-    hooks:
-      - id: seed-isort-config
   - repo: https://github.com/pycqa/isort
     rev: 5.12.0
     hooks:
@@ -50,3 +50,20 @@ repos:
         entry: .scripts/cleanup_placebo.sh
         require_serial: true
         files: "tests/*.json$"
+
+#  - repo: https://github.com/aws-cloudformation/cfn-lint
+#    rev: v0.78.1
+#    hooks:
+#      - id: cfn-lint
+#        files: .cicd/.*\.(json|yml|yaml)$
+
+  - repo: https://github.com/hadolint/hadolint
+    rev: v2.12.0
+    hooks:
+      - id: hadolint-docker
+        args: [ "-t", "error" ]
+
+  - repo: https://github.com/iamthefij/docker-pre-commit
+    rev: master
+    hooks:
+      - id: docker-compose-check

.scripts/import_sam_policies_template.py

@@ -1,4 +1,3 @@
-#   -*- coding: utf-8 -*-
 #  SPDX-License-Identifier: MPL-2.0
 #  Copyright 2020-2021 John Mille <john@compose-x.io>
 
@@ -62,7 +61,7 @@ def find_files(filename, search_path):
     TEMPLATE_FILE = find_files("policy_templates.json", ZIP_DIR.name)[0]
     DEST_FILE_PATH = path.abspath("ecs_composex/iam/sam_policies.json")
     print("Found source", TEMPLATE_FILE)
-    with open(TEMPLATE_FILE, "r") as file_fd:
+    with open(TEMPLATE_FILE) as file_fd:
         original_content = json.loads(file_fd.read())
     print("Outputting to", DEST_FILE_PATH)
     with open(DEST_FILE_PATH, "w") as file_fd:

NOTICES.rst

@@ -25,4 +25,3 @@ License files can be found here
 .. _Apache 2.0 License: https://www.apache.org/licenses/LICENSE-2.0.txt
 .. _BSD-2 Clause License: https://opensource.org/licenses/BSD-2-Clause
 .. _MIT license: https://opensource.org/licenses/MIT
-

docs/examples.rst

@@ -14,4 +14,3 @@ Examples and walk-through
 
     examples/lab
     examples/nginx_simple
-

e2e-examples/nginx_webapp/docker-compose.yaml

@@ -17,8 +17,8 @@ services:
     deploy:
       resources:
         limits:
-          cpus: 0.5
+          cpus: "0.5"
           memory: 256MB
         reservations:
-          cpus: 0.1
+          cpus: "0.1"
           memory: 128MB

use-cases/blog.features.yml

@@ -30,7 +30,7 @@ services:
         ecs.task.family: bignicefamily
       resources:
         reservations:
-          cpus: '0.25'
+          cpus: "0.25"
           memory: 1GB
     environment:
       LOGLEVEL: DEBUG