From dd2dc5deb0d86f1bf17cc4df887f5620bec98ffa Mon Sep 17 00:00:00 2001 From: Victor Seiji Hariki Date: Wed, 28 Dec 2022 21:38:10 -0300 Subject: [PATCH] Add test hook Signed-off-by: Victor Seiji Hariki --- .devtools/README.md | 12 ++++++++ lint.sh => .devtools/lint.sh | 0 .devtools/sethooks.sh | 3 ++ .githooks/.gitkeep | 0 .githooks/pre-commit | 49 ++++++++++++++++++++++++++++++ .github/workflows/cachebusting.yml | 22 ++++++++++++++ 6 files changed, 86 insertions(+) create mode 100644 .devtools/README.md rename lint.sh => .devtools/lint.sh (100%) mode change 100644 => 100755 create mode 100755 .devtools/sethooks.sh create mode 100644 .githooks/.gitkeep create mode 100755 .githooks/pre-commit create mode 100644 .github/workflows/cachebusting.yml diff --git a/.devtools/README.md b/.devtools/README.md new file mode 100644 index 0000000..22752a2 --- /dev/null +++ b/.devtools/README.md @@ -0,0 +1,12 @@ +# openOutpaint DevTools + +This is a folder containing some handy scripts to help developers to automate workflows and write code in openOutpaint's standards. +All scripts must be run from the root of the project. + +## `sethooks.sh` script + +This script will setup git hooks for this project. Hooks are mainly for cache busting purposes for now. It is recommended to run this script and setup hooks before any commits are made. + +## `lint.sh` script + +Uses `npm` to install prettier and lint javascript, html and css files according to `.prettierrc.json`. This script will install node modules locally, so editors with prettier support are recommended over this script. diff --git a/lint.sh b/.devtools/lint.sh old mode 100644 new mode 100755 similarity index 100% rename from lint.sh rename to .devtools/lint.sh diff --git a/.devtools/sethooks.sh b/.devtools/sethooks.sh new file mode 100755 index 0000000..7df28b7 --- /dev/null +++ b/.devtools/sethooks.sh @@ -0,0 +1,3 @@ +#!/usr/bin/sh + +git config core.hooksPath .githooks \ No newline at end of file diff --git a/.githooks/.gitkeep b/.githooks/.gitkeep new file mode 100644 index 0000000..e69de29 diff --git a/.githooks/pre-commit b/.githooks/pre-commit new file mode 100755 index 0000000..e144712 --- /dev/null +++ b/.githooks/pre-commit @@ -0,0 +1,49 @@ +#!/bin/sh +# +# An example hook script to verify what is about to be committed. +# Called by "git commit" with no arguments. The hook should +# exit with non-zero status after issuing an appropriate message if +# it wants to stop the commit. +# +# To enable this hook, rename this file to "pre-commit". + +if git rev-parse --verify HEAD >/dev/null 2>&1 +then + against=HEAD +else + # Initial commit: diff against an empty tree object + against=$(git hash-object -t tree /dev/null) +fi + +# If you want to allow non-ASCII filenames set this variable to true. +allownonascii=$(git config --type=bool hooks.allownonascii) + +# Redirect output to stderr. +exec 1>&2 + +# Cross platform projects tend to avoid non-ASCII filenames; prevent +# them from being added to the repository. We exploit the fact that the +# printable range starts at the space character and ends with tilde. +if [ "$allownonascii" != "true" ] && + # Note that the use of brackets around a tr range is ok here, (it's + # even required, for portability to Solaris 10's /usr/bin/tr), since + # the square bracket bytes happen to fall in the designated range. + test $(git diff --cached --name-only --diff-filter=A -z $against | + LC_ALL=C tr -d '[ -~]\0' | wc -c) != 0 +then + cat <<\EOF +Error: Attempt to add a non-ASCII file name. + +This can cause problems if you want to work with people on other platforms. + +To be portable it is advisable to rename the file. + +If you know what you are doing you can disable this check using: + + git config hooks.allownonascii true +EOF + exit 1 +fi + +# If there are whitespace errors, print the offending file names and fail. +exec git diff-index --check --cached $against -- diff --git a/.github/workflows/cachebusting.yml b/.github/workflows/cachebusting.yml new file mode 100644 index 0000000..b705004 --- /dev/null +++ b/.github/workflows/cachebusting.yml @@ -0,0 +1,22 @@ +name: Cache Busting +on: + push: + branches: + - "main" + - "testing" + pull_request: + branches: [main, testing] + types: [opened, synchronize, closed] + +jobs: + prettier: + runs-on: ubuntu-latest + steps: + - name: Checkout + uses: actions/checkout@v3 + with: + ref: ${{ github.event.pull_request.head.sha }} + - name: Prettify + uses: creyD/prettier_action@v4.2 + with: + prettier_options: --write **/*.{js,html,css,md}