hasura/graphql-engine
1
0
Fork 0
mirror of https://github.com/hasura/graphql-engine.git synced 2024-12-16 01:44:03 +03:00
Code Issues Projects Releases Wiki Activity
c90b8c4776
graphql-engine/Makefile

134 lines
3.9 KiB
Makefile
Raw Normal View History

tooling: add test commands to Makefile PR-URL: https://github.com/hasura/graphql-engine-mono/pull/4787 GitOrigin-RevId: 6fef3933dbc414ddc79f0f738e91e28bee4b8c4f
2022-06-20 16:33:04 +03:00
include Makefile.tests
server, CI: use ormolu as a formatter for Haskell sources ### Description - sets up a Makefile target for running ormolu to format and check source code - updates CI to run ormolu instead of stylish-haskell (and to check instead of format actively) Compare #1679. Here's the plan for merging this: 1. merge this PR; at this point, all PRs will fail CI unless they have the `ignore-server-format-checks` label set 2. merge follow-up PR #2404 that does nothing but actually reformats the codebase 3. tag the merge commit as `post-ormolu` (also on `graphql-engine`, for the benefits of community contributors) 4. provide the following script to any devs in order to update their branches: ``` $ git checkout my-feature-branch $ git merge post-ormolu^ $ make format $ git commit -a -m "reformat with ormolu" $ git merge -s ours post-ormolu ``` (I'll put this in the commit message) https://github.com/hasura/graphql-engine-mono/pull/2020 Co-authored-by: Philip Lykke Carlsen <358550+plcplc@users.noreply.github.com> Co-authored-by: Swann Moreau <62569634+evertedsphere@users.noreply.github.com> GitOrigin-RevId: 130f480a6d79967c8d045b7f3a6dec30b10472a7
2021-09-24 00:22:27 +03:00
# skip contrib with its generated .hs file because it doesn't
# come with a cabal file, which can trigger a bug in ormolu
Add Makefile targets to run `hlint` on all and changed files. * `make lint-hs` will lint all files. * `make lint-hs-changed` will lint all modified files on this branch. * `make lint` will delegate to `make lint-hs`. * `make lint-changed` will delegate to `make lint-hs-changed`. PR-URL: https://github.com/hasura/graphql-engine-mono/pull/4720 GitOrigin-RevId: d872f5b672b1d06100cd9fdaa5e60dc8bb5cae86
2022-06-14 18:20:12 +03:00
HS_FILES = $(shell git ls-files '*.hs' '*.hs-boot' | grep -v '^contrib/')
Makefile: Run `ormolu --check` and `shellcheck` as part of linting. In the interest of having a single command to run before committing in order to check for basic errors, I propose `make lint-changed`. This enhances `make lint` and `make lint-changed` to run `ormolu` and `shellcheck`, in line with CI. PR-URL: https://github.com/hasura/graphql-engine-mono/pull/4727 GitOrigin-RevId: 05c860a86ce40b0f52c6aa2d333c0af23aea070e
2022-06-15 13:26:00 +03:00
CHANGED_HS_FILES = $(shell git diff --diff-filter=d --name-only `git merge-base HEAD origin/main` | grep '.*\(\.hs\|hs-boot\)$$' | grep -v '^contrib/')
Nix: Use `nixpkgs-fmt` to format everything. You can now run `nix fmt` (or `make format-changed`) to reformat Nix files. This is not enforced by CI. PR-URL: https://github.com/hasura/graphql-engine-mono/pull/4754 GitOrigin-RevId: a2e7cbe6c037d68ba6303278616314de60b6aa72
2022-06-20 16:01:37 +03:00
NIX_FILES = $(shell git ls-files '*.nix' 'nix/*.nix')
Makefile: Run `ormolu --check` and `shellcheck` as part of linting. In the interest of having a single command to run before committing in order to check for basic errors, I propose `make lint-changed`. This enhances `make lint` and `make lint-changed` to run `ormolu` and `shellcheck`, in line with CI. PR-URL: https://github.com/hasura/graphql-engine-mono/pull/4727 GitOrigin-RevId: 05c860a86ce40b0f52c6aa2d333c0af23aea070e
2022-06-15 13:26:00 +03:00
SHELL_FILES = $(shell git ls-files '*.sh')
CHANGED_SHELL_FILES = $(shell git diff --diff-filter=d --name-only `git merge-base HEAD origin/main` | grep '.*\.sh$$')
Add Makefile targets to run `hlint` on all and changed files. * `make lint-hs` will lint all files. * `make lint-hs-changed` will lint all modified files on this branch. * `make lint` will delegate to `make lint-hs`. * `make lint-changed` will delegate to `make lint-hs-changed`. PR-URL: https://github.com/hasura/graphql-engine-mono/pull/4720 GitOrigin-RevId: d872f5b672b1d06100cd9fdaa5e60dc8bb5cae86
2022-06-14 18:20:12 +03:00
HLINT = hlint
server, CI: use ormolu as a formatter for Haskell sources ### Description - sets up a Makefile target for running ormolu to format and check source code - updates CI to run ormolu instead of stylish-haskell (and to check instead of format actively) Compare #1679. Here's the plan for merging this: 1. merge this PR; at this point, all PRs will fail CI unless they have the `ignore-server-format-checks` label set 2. merge follow-up PR #2404 that does nothing but actually reformats the codebase 3. tag the merge commit as `post-ormolu` (also on `graphql-engine`, for the benefits of community contributors) 4. provide the following script to any devs in order to update their branches: ``` $ git checkout my-feature-branch $ git merge post-ormolu^ $ make format $ git commit -a -m "reformat with ormolu" $ git merge -s ours post-ormolu ``` (I'll put this in the commit message) https://github.com/hasura/graphql-engine-mono/pull/2020 Co-authored-by: Philip Lykke Carlsen <358550+plcplc@users.noreply.github.com> Co-authored-by: Swann Moreau <62569634+evertedsphere@users.noreply.github.com> GitOrigin-RevId: 130f480a6d79967c8d045b7f3a6dec30b10472a7
2021-09-24 00:22:27 +03:00
Nix: Use `nixpkgs-fmt` to format everything. You can now run `nix fmt` (or `make format-changed`) to reformat Nix files. This is not enforced by CI. PR-URL: https://github.com/hasura/graphql-engine-mono/pull/4754 GitOrigin-RevId: a2e7cbe6c037d68ba6303278616314de60b6aa72
2022-06-20 16:01:37 +03:00
NIX_FMT = nixpkgs-fmt
server, CI: use ormolu as a formatter for Haskell sources ### Description - sets up a Makefile target for running ormolu to format and check source code - updates CI to run ormolu instead of stylish-haskell (and to check instead of format actively) Compare #1679. Here's the plan for merging this: 1. merge this PR; at this point, all PRs will fail CI unless they have the `ignore-server-format-checks` label set 2. merge follow-up PR #2404 that does nothing but actually reformats the codebase 3. tag the merge commit as `post-ormolu` (also on `graphql-engine`, for the benefits of community contributors) 4. provide the following script to any devs in order to update their branches: ``` $ git checkout my-feature-branch $ git merge post-ormolu^ $ make format $ git commit -a -m "reformat with ormolu" $ git merge -s ours post-ormolu ``` (I'll put this in the commit message) https://github.com/hasura/graphql-engine-mono/pull/2020 Co-authored-by: Philip Lykke Carlsen <358550+plcplc@users.noreply.github.com> Co-authored-by: Swann Moreau <62569634+evertedsphere@users.noreply.github.com> GitOrigin-RevId: 130f480a6d79967c8d045b7f3a6dec30b10472a7
2021-09-24 00:22:27 +03:00
ORMOLU_CHECK_VERSION = 0.3.0.0
ORMOLU_ARGS = --cabal-default-extensions
ORMOLU = ormolu
ORMOLU_VERSION = $(shell $(ORMOLU) --version | awk 'NR==1 { print $$2 }')
Makefile: Run `ormolu --check` and `shellcheck` as part of linting. In the interest of having a single command to run before committing in order to check for basic errors, I propose `make lint-changed`. This enhances `make lint` and `make lint-changed` to run `ormolu` and `shellcheck`, in line with CI. PR-URL: https://github.com/hasura/graphql-engine-mono/pull/4727 GitOrigin-RevId: 05c860a86ce40b0f52c6aa2d333c0af23aea070e
2022-06-15 13:26:00 +03:00
SHELLCHECK = shellcheck
server, CI: use ormolu as a formatter for Haskell sources ### Description - sets up a Makefile target for running ormolu to format and check source code - updates CI to run ormolu instead of stylish-haskell (and to check instead of format actively) Compare #1679. Here's the plan for merging this: 1. merge this PR; at this point, all PRs will fail CI unless they have the `ignore-server-format-checks` label set 2. merge follow-up PR #2404 that does nothing but actually reformats the codebase 3. tag the merge commit as `post-ormolu` (also on `graphql-engine`, for the benefits of community contributors) 4. provide the following script to any devs in order to update their branches: ``` $ git checkout my-feature-branch $ git merge post-ormolu^ $ make format $ git commit -a -m "reformat with ormolu" $ git merge -s ours post-ormolu ``` (I'll put this in the commit message) https://github.com/hasura/graphql-engine-mono/pull/2020 Co-authored-by: Philip Lykke Carlsen <358550+plcplc@users.noreply.github.com> Co-authored-by: Swann Moreau <62569634+evertedsphere@users.noreply.github.com> GitOrigin-RevId: 130f480a6d79967c8d045b7f3a6dec30b10472a7
2021-09-24 00:22:27 +03:00
# default target
.PHONY: help
## help: prints help message
help:
@echo "Usage:"
@sed -n 's/^##//p' ${MAKEFILE_LIST} | column -t -s ':' | sed -e 's/^/ /'
.PHONY: check-ormolu-version
check-ormolu-version:
@if ! [ "$(ORMOLU_VERSION)" = "$(ORMOLU_CHECK_VERSION)" ]; then \
echo "WARNING: ormolu version mismatch, expected $(ORMOLU_CHECK_VERSION)"; \
fi
.PHONY: format-hs
## format-hs: auto-format Haskell source code using ormolu
format-hs: check-ormolu-version
@echo running ormolu --mode inplace
Add Makefile targets to run `hlint` on all and changed files. * `make lint-hs` will lint all files. * `make lint-hs-changed` will lint all modified files on this branch. * `make lint` will delegate to `make lint-hs`. * `make lint-changed` will delegate to `make lint-hs-changed`. PR-URL: https://github.com/hasura/graphql-engine-mono/pull/4720 GitOrigin-RevId: d872f5b672b1d06100cd9fdaa5e60dc8bb5cae86
2022-06-14 18:20:12 +03:00
@$(ORMOLU) $(ORMOLU_ARGS) --mode inplace $(HS_FILES)
server, CI: use ormolu as a formatter for Haskell sources ### Description - sets up a Makefile target for running ormolu to format and check source code - updates CI to run ormolu instead of stylish-haskell (and to check instead of format actively) Compare #1679. Here's the plan for merging this: 1. merge this PR; at this point, all PRs will fail CI unless they have the `ignore-server-format-checks` label set 2. merge follow-up PR #2404 that does nothing but actually reformats the codebase 3. tag the merge commit as `post-ormolu` (also on `graphql-engine`, for the benefits of community contributors) 4. provide the following script to any devs in order to update their branches: ``` $ git checkout my-feature-branch $ git merge post-ormolu^ $ make format $ git commit -a -m "reformat with ormolu" $ git merge -s ours post-ormolu ``` (I'll put this in the commit message) https://github.com/hasura/graphql-engine-mono/pull/2020 Co-authored-by: Philip Lykke Carlsen <358550+plcplc@users.noreply.github.com> Co-authored-by: Swann Moreau <62569634+evertedsphere@users.noreply.github.com> GitOrigin-RevId: 130f480a6d79967c8d045b7f3a6dec30b10472a7
2021-09-24 00:22:27 +03:00
.PHONY: format-hs-changed
## format-hs-changed: auto-format Haskell source code using ormolu (changed files only)
format-hs-changed: check-ormolu-version
@echo running ormolu --mode inplace
Add Makefile targets to run `hlint` on all and changed files. * `make lint-hs` will lint all files. * `make lint-hs-changed` will lint all modified files on this branch. * `make lint` will delegate to `make lint-hs`. * `make lint-changed` will delegate to `make lint-hs-changed`. PR-URL: https://github.com/hasura/graphql-engine-mono/pull/4720 GitOrigin-RevId: d872f5b672b1d06100cd9fdaa5e60dc8bb5cae86
2022-06-14 18:20:12 +03:00
@if [ -n "$(CHANGED_HS_FILES)" ]; then \
$(ORMOLU) $(ORMOLU_ARGS) --mode inplace $(CHANGED_HS_FILES); \
server, CI: use ormolu as a formatter for Haskell sources ### Description - sets up a Makefile target for running ormolu to format and check source code - updates CI to run ormolu instead of stylish-haskell (and to check instead of format actively) Compare #1679. Here's the plan for merging this: 1. merge this PR; at this point, all PRs will fail CI unless they have the `ignore-server-format-checks` label set 2. merge follow-up PR #2404 that does nothing but actually reformats the codebase 3. tag the merge commit as `post-ormolu` (also on `graphql-engine`, for the benefits of community contributors) 4. provide the following script to any devs in order to update their branches: ``` $ git checkout my-feature-branch $ git merge post-ormolu^ $ make format $ git commit -a -m "reformat with ormolu" $ git merge -s ours post-ormolu ``` (I'll put this in the commit message) https://github.com/hasura/graphql-engine-mono/pull/2020 Co-authored-by: Philip Lykke Carlsen <358550+plcplc@users.noreply.github.com> Co-authored-by: Swann Moreau <62569634+evertedsphere@users.noreply.github.com> GitOrigin-RevId: 130f480a6d79967c8d045b7f3a6dec30b10472a7
2021-09-24 00:22:27 +03:00
fi
.PHONY: check-format-hs
## check-format-hs: check Haskell source code formatting using ormolu
check-format-hs: check-ormolu-version
@echo running ormolu --mode check
Add Makefile targets to run `hlint` on all and changed files. * `make lint-hs` will lint all files. * `make lint-hs-changed` will lint all modified files on this branch. * `make lint` will delegate to `make lint-hs`. * `make lint-changed` will delegate to `make lint-hs-changed`. PR-URL: https://github.com/hasura/graphql-engine-mono/pull/4720 GitOrigin-RevId: d872f5b672b1d06100cd9fdaa5e60dc8bb5cae86
2022-06-14 18:20:12 +03:00
@$(ORMOLU) $(ORMOLU_ARGS) --mode check $(HS_FILES)
server, CI: use ormolu as a formatter for Haskell sources ### Description - sets up a Makefile target for running ormolu to format and check source code - updates CI to run ormolu instead of stylish-haskell (and to check instead of format actively) Compare #1679. Here's the plan for merging this: 1. merge this PR; at this point, all PRs will fail CI unless they have the `ignore-server-format-checks` label set 2. merge follow-up PR #2404 that does nothing but actually reformats the codebase 3. tag the merge commit as `post-ormolu` (also on `graphql-engine`, for the benefits of community contributors) 4. provide the following script to any devs in order to update their branches: ``` $ git checkout my-feature-branch $ git merge post-ormolu^ $ make format $ git commit -a -m "reformat with ormolu" $ git merge -s ours post-ormolu ``` (I'll put this in the commit message) https://github.com/hasura/graphql-engine-mono/pull/2020 Co-authored-by: Philip Lykke Carlsen <358550+plcplc@users.noreply.github.com> Co-authored-by: Swann Moreau <62569634+evertedsphere@users.noreply.github.com> GitOrigin-RevId: 130f480a6d79967c8d045b7f3a6dec30b10472a7
2021-09-24 00:22:27 +03:00
Makefile: Run `ormolu --check` and `shellcheck` as part of linting. In the interest of having a single command to run before committing in order to check for basic errors, I propose `make lint-changed`. This enhances `make lint` and `make lint-changed` to run `ormolu` and `shellcheck`, in line with CI. PR-URL: https://github.com/hasura/graphql-engine-mono/pull/4727 GitOrigin-RevId: 05c860a86ce40b0f52c6aa2d333c0af23aea070e
2022-06-15 13:26:00 +03:00
.PHONY: check-format-hs-changed
## check-format-hs-changed: check Haskell source code formatting using ormolu (changed-files-only)
check-format-hs-changed: check-ormolu-version
@echo running ormolu --mode check
@if [ -n "$(CHANGED_HS_FILES)" ]; then \
$(ORMOLU) $(ORMOLU_ARGS) --mode check $(CHANGED_HS_FILES); \
fi
Nix: Use `nixpkgs-fmt` to format everything. You can now run `nix fmt` (or `make format-changed`) to reformat Nix files. This is not enforced by CI. PR-URL: https://github.com/hasura/graphql-engine-mono/pull/4754 GitOrigin-RevId: a2e7cbe6c037d68ba6303278616314de60b6aa72
2022-06-20 16:01:37 +03:00
# We don't bother checking only changed *.nix files, as there's so few.
.PHONY: format-nix
## format-nix: auto-format Nix source code using `nixpkgs-fmt`
format-nix:
@if command -v $(NIX_FMT) > /dev/null; then \
echo "running $(NIX_FMT)"; \
$(NIX_FMT) $(NIX_FILES); \
else \
echo "$(NIX_FMT) is not installed; skipping"; \
fi
.PHONY: check-format-nix
## check-format-nix: check Nix source code using `nixpkgs-fmt`
check-format-nix:
@if command -v $(NIX_FMT) > /dev/null; then \
echo "running $(NIX_FMT) --check"; \
$(NIX_FMT) --check $(NIX_FILES); \
else \
echo "$(NIX_FMT) is not installed; skipping"; \
fi
server, CI: use ormolu as a formatter for Haskell sources ### Description - sets up a Makefile target for running ormolu to format and check source code - updates CI to run ormolu instead of stylish-haskell (and to check instead of format actively) Compare #1679. Here's the plan for merging this: 1. merge this PR; at this point, all PRs will fail CI unless they have the `ignore-server-format-checks` label set 2. merge follow-up PR #2404 that does nothing but actually reformats the codebase 3. tag the merge commit as `post-ormolu` (also on `graphql-engine`, for the benefits of community contributors) 4. provide the following script to any devs in order to update their branches: ``` $ git checkout my-feature-branch $ git merge post-ormolu^ $ make format $ git commit -a -m "reformat with ormolu" $ git merge -s ours post-ormolu ``` (I'll put this in the commit message) https://github.com/hasura/graphql-engine-mono/pull/2020 Co-authored-by: Philip Lykke Carlsen <358550+plcplc@users.noreply.github.com> Co-authored-by: Swann Moreau <62569634+evertedsphere@users.noreply.github.com> GitOrigin-RevId: 130f480a6d79967c8d045b7f3a6dec30b10472a7
2021-09-24 00:22:27 +03:00
.PHONY: format
Nix: Use `nixpkgs-fmt` to format everything. You can now run `nix fmt` (or `make format-changed`) to reformat Nix files. This is not enforced by CI. PR-URL: https://github.com/hasura/graphql-engine-mono/pull/4754 GitOrigin-RevId: a2e7cbe6c037d68ba6303278616314de60b6aa72
2022-06-20 16:01:37 +03:00
format: format-hs format-nix
server, CI: use ormolu as a formatter for Haskell sources ### Description - sets up a Makefile target for running ormolu to format and check source code - updates CI to run ormolu instead of stylish-haskell (and to check instead of format actively) Compare #1679. Here's the plan for merging this: 1. merge this PR; at this point, all PRs will fail CI unless they have the `ignore-server-format-checks` label set 2. merge follow-up PR #2404 that does nothing but actually reformats the codebase 3. tag the merge commit as `post-ormolu` (also on `graphql-engine`, for the benefits of community contributors) 4. provide the following script to any devs in order to update their branches: ``` $ git checkout my-feature-branch $ git merge post-ormolu^ $ make format $ git commit -a -m "reformat with ormolu" $ git merge -s ours post-ormolu ``` (I'll put this in the commit message) https://github.com/hasura/graphql-engine-mono/pull/2020 Co-authored-by: Philip Lykke Carlsen <358550+plcplc@users.noreply.github.com> Co-authored-by: Swann Moreau <62569634+evertedsphere@users.noreply.github.com> GitOrigin-RevId: 130f480a6d79967c8d045b7f3a6dec30b10472a7
2021-09-24 00:22:27 +03:00
.PHONY: format-changed
Nix: Use `nixpkgs-fmt` to format everything. You can now run `nix fmt` (or `make format-changed`) to reformat Nix files. This is not enforced by CI. PR-URL: https://github.com/hasura/graphql-engine-mono/pull/4754 GitOrigin-RevId: a2e7cbe6c037d68ba6303278616314de60b6aa72
2022-06-20 16:01:37 +03:00
format-changed: format-hs-changed format-nix
server, CI: use ormolu as a formatter for Haskell sources ### Description - sets up a Makefile target for running ormolu to format and check source code - updates CI to run ormolu instead of stylish-haskell (and to check instead of format actively) Compare #1679. Here's the plan for merging this: 1. merge this PR; at this point, all PRs will fail CI unless they have the `ignore-server-format-checks` label set 2. merge follow-up PR #2404 that does nothing but actually reformats the codebase 3. tag the merge commit as `post-ormolu` (also on `graphql-engine`, for the benefits of community contributors) 4. provide the following script to any devs in order to update their branches: ``` $ git checkout my-feature-branch $ git merge post-ormolu^ $ make format $ git commit -a -m "reformat with ormolu" $ git merge -s ours post-ormolu ``` (I'll put this in the commit message) https://github.com/hasura/graphql-engine-mono/pull/2020 Co-authored-by: Philip Lykke Carlsen <358550+plcplc@users.noreply.github.com> Co-authored-by: Swann Moreau <62569634+evertedsphere@users.noreply.github.com> GitOrigin-RevId: 130f480a6d79967c8d045b7f3a6dec30b10472a7
2021-09-24 00:22:27 +03:00
.PHONY: check-format
Nix: Use `nixpkgs-fmt` to format everything. You can now run `nix fmt` (or `make format-changed`) to reformat Nix files. This is not enforced by CI. PR-URL: https://github.com/hasura/graphql-engine-mono/pull/4754 GitOrigin-RevId: a2e7cbe6c037d68ba6303278616314de60b6aa72
2022-06-20 16:01:37 +03:00
check-format: check-format-hs check-format-nix
Add Makefile targets to run `hlint` on all and changed files. * `make lint-hs` will lint all files. * `make lint-hs-changed` will lint all modified files on this branch. * `make lint` will delegate to `make lint-hs`. * `make lint-changed` will delegate to `make lint-hs-changed`. PR-URL: https://github.com/hasura/graphql-engine-mono/pull/4720 GitOrigin-RevId: d872f5b672b1d06100cd9fdaa5e60dc8bb5cae86
2022-06-14 18:20:12 +03:00
Makefile: Run `ormolu --check` and `shellcheck` as part of linting. In the interest of having a single command to run before committing in order to check for basic errors, I propose `make lint-changed`. This enhances `make lint` and `make lint-changed` to run `ormolu` and `shellcheck`, in line with CI. PR-URL: https://github.com/hasura/graphql-engine-mono/pull/4727 GitOrigin-RevId: 05c860a86ce40b0f52c6aa2d333c0af23aea070e
2022-06-15 13:26:00 +03:00
.PHONY: check-format-changed
Nix: Use `nixpkgs-fmt` to format everything. You can now run `nix fmt` (or `make format-changed`) to reformat Nix files. This is not enforced by CI. PR-URL: https://github.com/hasura/graphql-engine-mono/pull/4754 GitOrigin-RevId: a2e7cbe6c037d68ba6303278616314de60b6aa72
2022-06-20 16:01:37 +03:00
check-format-changed: check-format-hs-changed check-format-nix
Makefile: Run `ormolu --check` and `shellcheck` as part of linting. In the interest of having a single command to run before committing in order to check for basic errors, I propose `make lint-changed`. This enhances `make lint` and `make lint-changed` to run `ormolu` and `shellcheck`, in line with CI. PR-URL: https://github.com/hasura/graphql-engine-mono/pull/4727 GitOrigin-RevId: 05c860a86ce40b0f52c6aa2d333c0af23aea070e
2022-06-15 13:26:00 +03:00
Add Makefile targets to run `hlint` on all and changed files. * `make lint-hs` will lint all files. * `make lint-hs-changed` will lint all modified files on this branch. * `make lint` will delegate to `make lint-hs`. * `make lint-changed` will delegate to `make lint-hs-changed`. PR-URL: https://github.com/hasura/graphql-engine-mono/pull/4720 GitOrigin-RevId: d872f5b672b1d06100cd9fdaa5e60dc8bb5cae86
2022-06-14 18:20:12 +03:00
.PHONY: lint-hs
## lint-hs: lint Haskell code using `hlint`
lint-hs:
@echo running hlint
@$(HLINT) $(HS_FILES)
.PHONY: lint-hs-changed
## lint-hs-changed: lint Haskell code using `hlint` (changed files only)
lint-hs-changed:
@echo running hlint
@if [ -n "$(CHANGED_HS_FILES)" ]; then \
$(HLINT) $(CHANGED_HS_FILES); \
fi
Makefile: Run `ormolu --check` and `shellcheck` as part of linting. In the interest of having a single command to run before committing in order to check for basic errors, I propose `make lint-changed`. This enhances `make lint` and `make lint-changed` to run `ormolu` and `shellcheck`, in line with CI. PR-URL: https://github.com/hasura/graphql-engine-mono/pull/4727 GitOrigin-RevId: 05c860a86ce40b0f52c6aa2d333c0af23aea070e
2022-06-15 13:26:00 +03:00
.PHONY: lint-shell
Nix: Use `nixpkgs-fmt` to format everything. You can now run `nix fmt` (or `make format-changed`) to reformat Nix files. This is not enforced by CI. PR-URL: https://github.com/hasura/graphql-engine-mono/pull/4754 GitOrigin-RevId: a2e7cbe6c037d68ba6303278616314de60b6aa72
2022-06-20 16:01:37 +03:00
## lint-shell: lint shell scripts using `shellcheck`
Makefile: Run `ormolu --check` and `shellcheck` as part of linting. In the interest of having a single command to run before committing in order to check for basic errors, I propose `make lint-changed`. This enhances `make lint` and `make lint-changed` to run `ormolu` and `shellcheck`, in line with CI. PR-URL: https://github.com/hasura/graphql-engine-mono/pull/4727 GitOrigin-RevId: 05c860a86ce40b0f52c6aa2d333c0af23aea070e
2022-06-15 13:26:00 +03:00
lint-shell:
@echo running shellcheck
@$(SHELLCHECK) $(SHELL_FILES)
.PHONY: lint-shell-changed
Nix: Use `nixpkgs-fmt` to format everything. You can now run `nix fmt` (or `make format-changed`) to reformat Nix files. This is not enforced by CI. PR-URL: https://github.com/hasura/graphql-engine-mono/pull/4754 GitOrigin-RevId: a2e7cbe6c037d68ba6303278616314de60b6aa72
2022-06-20 16:01:37 +03:00
## lint-shell-changed: lint shell scripts using `shellcheck` (changed files only)
Makefile: Run `ormolu --check` and `shellcheck` as part of linting. In the interest of having a single command to run before committing in order to check for basic errors, I propose `make lint-changed`. This enhances `make lint` and `make lint-changed` to run `ormolu` and `shellcheck`, in line with CI. PR-URL: https://github.com/hasura/graphql-engine-mono/pull/4727 GitOrigin-RevId: 05c860a86ce40b0f52c6aa2d333c0af23aea070e
2022-06-15 13:26:00 +03:00
lint-shell-changed:
@echo running shellcheck
@if [ -n "$(CHANGED_SHELL_FILES)" ]; then \
$(SHELLCHECK) $(CHANGED_SHELL_FILES); \
fi
Add Makefile targets to run `hlint` on all and changed files. * `make lint-hs` will lint all files. * `make lint-hs-changed` will lint all modified files on this branch. * `make lint` will delegate to `make lint-hs`. * `make lint-changed` will delegate to `make lint-hs-changed`. PR-URL: https://github.com/hasura/graphql-engine-mono/pull/4720 GitOrigin-RevId: d872f5b672b1d06100cd9fdaa5e60dc8bb5cae86
2022-06-14 18:20:12 +03:00
.PHONY: lint
Makefile: Run `ormolu --check` and `shellcheck` as part of linting. In the interest of having a single command to run before committing in order to check for basic errors, I propose `make lint-changed`. This enhances `make lint` and `make lint-changed` to run `ormolu` and `shellcheck`, in line with CI. PR-URL: https://github.com/hasura/graphql-engine-mono/pull/4727 GitOrigin-RevId: 05c860a86ce40b0f52c6aa2d333c0af23aea070e
2022-06-15 13:26:00 +03:00
lint: lint-hs lint-shell check-format
Add Makefile targets to run `hlint` on all and changed files. * `make lint-hs` will lint all files. * `make lint-hs-changed` will lint all modified files on this branch. * `make lint` will delegate to `make lint-hs`. * `make lint-changed` will delegate to `make lint-hs-changed`. PR-URL: https://github.com/hasura/graphql-engine-mono/pull/4720 GitOrigin-RevId: d872f5b672b1d06100cd9fdaa5e60dc8bb5cae86
2022-06-14 18:20:12 +03:00
.PHONY: lint-changed
Makefile: Run `ormolu --check` and `shellcheck` as part of linting. In the interest of having a single command to run before committing in order to check for basic errors, I propose `make lint-changed`. This enhances `make lint` and `make lint-changed` to run `ormolu` and `shellcheck`, in line with CI. PR-URL: https://github.com/hasura/graphql-engine-mono/pull/4727 GitOrigin-RevId: 05c860a86ce40b0f52c6aa2d333c0af23aea070e
2022-06-15 13:26:00 +03:00
lint-changed: lint-hs-changed lint-shell-changed check-format-changed
tooling: add test commands to Makefile PR-URL: https://github.com/hasura/graphql-engine-mono/pull/4787 GitOrigin-RevId: 6fef3933dbc414ddc79f0f738e91e28bee4b8c4f
2022-06-20 16:33:04 +03:00
Reference in New Issue Copy Permalink