Urban-Analytics-Technology-Platform / web-app-template

https://urban-analytics-technology-platform.github.io/web-app-template/
2 stars 0 forks source link

Minimal test and `.pre-commit` CI workflow #20

Closed griff-rees closed 1 month ago

griff-rees commented 1 month ago

Addresses #14.

What it doesn't do (yet):

griff-rees commented 1 month ago

@dabreegster: hoping this is minimal enough for your liking via @yongrenjie's suggestion of simply not init-ing pre-commit locally. This leaves your local checkout free to commit without checks. You can then use pre-commit run --all-files whenever like below:

$ cd path/to/checkout/web-app-template
$ pre-commit run --all-files
check for merge conflicts................................................Passed
check toml...........................................(no files to check)Skipped
check for case conflicts.................................................Passed
debug statements (python)............................(no files to check)Skipped
fix end of files.........................................................Passed
trim trailing whitespace.................................................Passed
detect private key.......................................................Passed
mixed line ending........................................................Passed
sort simple yaml files...............................(no files to check)Skipped
check that scripts with shebangs are executable..........................Passed
check for case conflicts.................................................Passed
check for broken symlinks............................(no files to check)Skipped
prettier-format..........................................................Passed
dabreegster commented 1 month ago

Could you rebase please? I think I also fixed the generated files getting checked into git

griff-rees commented 1 month ago

@dabreegster I've rebased. I'll update the pull request description as well.