Skip to content
Snippets Groups Projects
Unverified Commit 4d85411b authored by mergify[bot]'s avatar mergify[bot] Committed by GitHub
Browse files

Merge pull request #9429 from mpickering/wip/style-commit

formatting: Add style-commit makefile target
parents eaa5245e 4c884c0a
No related branches found
No related tags found
No related merge requests found
Pipeline #86997 passed
......@@ -155,8 +155,15 @@ Code Style
---------------
We use automated formatting with Fourmolu to enforce a unified style across the code bases. It is checked in the CI process.
After installing Fourmolu 0.12, you can automatically format the code bases with `make style` at the top level of the project.
You can also use `make style-modified` to only format modified files.
After installing Fourmolu 0.12, there are some makefile targets to help formatting
the code base.
* `make style` - Format the `Cabal`, `Cabal-syntax` and `cabal-install` directories.
* `make style-modified` - Format files modified in the current tree.
* `make style-commit COMMIT=<ref>` - Format files modified between HEAD and the given reference.
Other Conventions
-----------------
......
......@@ -25,6 +25,10 @@ style-modified: ## Run the code styler on modified files
@git ls-files --modified Cabal Cabal-syntax cabal-install \
| grep '.hs$$' | xargs -P $(PROCS) -I {} fourmolu -q -i {}
style-commit: ## Run the code styler on the previous commit
@git diff --name-only HEAD $(COMMIT) Cabal Cabal-syntax cabal-install \
| grep '.hs$$' | xargs -P $(PROCS) -I {} fourmolu -q -i {}
# source generation: SPDX
SPDX_LICENSE_HS:=Cabal-syntax/src/Distribution/SPDX/LicenseId.hs
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment