2024-03-27 22:48:59 +00:00
|
|
|
# This file was copied mostly from check-maintainers-sorted.yaml.
|
|
|
|
# NOTE: Formatting with the RFC-style nixfmt command is not yet stable. See
|
|
|
|
# https://github.com/NixOS/rfcs/pull/166.
|
|
|
|
# Because of this, this action is not yet enabled for all files -- only for
|
|
|
|
# those who have opted in.
|
|
|
|
name: Check that Nix files are formatted
|
|
|
|
|
|
|
|
on:
|
|
|
|
pull_request_target:
|
2024-07-23 20:03:15 +01:00
|
|
|
# See the comment at the same location in ./check-by-name.yml
|
|
|
|
types: [opened, synchronize, reopened, edited]
|
2024-03-27 22:48:59 +00:00
|
|
|
permissions:
|
|
|
|
contents: read
|
|
|
|
|
|
|
|
jobs:
|
|
|
|
nixos:
|
|
|
|
runs-on: ubuntu-latest
|
2024-07-23 20:03:15 +01:00
|
|
|
if: "github.repository_owner == 'NixOS' && !contains(github.event.pull_request.title, '[skip treewide]')"
|
2024-03-27 22:48:59 +00:00
|
|
|
steps:
|
2024-06-29 23:01:52 +01:00
|
|
|
- uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332 # v4.1.7
|
2024-03-27 22:48:59 +00:00
|
|
|
with:
|
|
|
|
# pull_request_target checks out the base branch by default
|
|
|
|
ref: refs/pull/${{ github.event.pull_request.number }}/merge
|
2024-07-23 20:03:15 +01:00
|
|
|
# Fetches the merge commit and its parents
|
|
|
|
fetch-depth: 2
|
|
|
|
- name: Checking out base branch
|
|
|
|
run: |
|
|
|
|
base=$(mktemp -d)
|
|
|
|
baseRev=$(git rev-parse HEAD^1)
|
|
|
|
git worktree add "$base" "$baseRev"
|
|
|
|
echo "baseRev=$baseRev" >> "$GITHUB_ENV"
|
|
|
|
echo "base=$base" >> "$GITHUB_ENV"
|
2024-06-25 23:52:13 +01:00
|
|
|
- name: Get Nixpkgs revision for nixfmt
|
|
|
|
run: |
|
|
|
|
# pin to a commit from nixpkgs-unstable to avoid e.g. building nixfmt
|
|
|
|
# from staging
|
|
|
|
# This should not be a URL, because it would allow PRs to run arbitrary code in CI!
|
|
|
|
rev=$(jq -r .rev ci/pinned-nixpkgs.json)
|
|
|
|
echo "url=https://github.com/NixOS/nixpkgs/archive/$rev.tar.gz" >> "$GITHUB_ENV"
|
2024-07-22 21:04:12 +01:00
|
|
|
- uses: cachix/install-nix-action@ba0dd844c9180cbf77aa72a116d6fbc515d0e87b # v27
|
2024-03-27 22:48:59 +00:00
|
|
|
with:
|
|
|
|
# explicitly enable sandbox
|
|
|
|
extra_nix_config: sandbox = true
|
2024-06-25 23:52:13 +01:00
|
|
|
nix_path: nixpkgs=${{ env.url }}
|
2024-03-27 22:48:59 +00:00
|
|
|
- name: Install nixfmt
|
2024-04-03 18:57:03 +01:00
|
|
|
run: "nix-env -f '<nixpkgs>' -iAP nixfmt-rfc-style"
|
2024-03-27 22:48:59 +00:00
|
|
|
- name: Check that Nix files are formatted according to the RFC style
|
|
|
|
run: |
|
2024-07-23 20:03:15 +01:00
|
|
|
unformattedFiles=()
|
|
|
|
|
|
|
|
# TODO: Make this more parallel
|
|
|
|
|
|
|
|
# Loop through all Nix files touched by the PR
|
|
|
|
while readarray -d '' -n 2 entry && (( ${#entry[@]} != 0 )); do
|
|
|
|
type=${entry[0]}
|
|
|
|
file=${entry[1]}
|
|
|
|
case $type in
|
|
|
|
A*)
|
|
|
|
source=""
|
|
|
|
dest=$file
|
|
|
|
;;
|
|
|
|
M*)
|
|
|
|
source=$file
|
|
|
|
dest=$file
|
|
|
|
;;
|
|
|
|
C*|R*)
|
|
|
|
source=$file
|
|
|
|
read -r -d '' dest
|
|
|
|
;;
|
|
|
|
*)
|
|
|
|
echo "Ignoring file $file with type $type"
|
|
|
|
continue
|
|
|
|
esac
|
|
|
|
|
|
|
|
# Ignore files that weren't already formatted
|
|
|
|
if [[ -n "$source" ]] && ! nixfmt --check ${{ env.base }}/"$source" 2>/dev/null; then
|
|
|
|
echo "Ignoring file $file because it's not formatted in the base commit"
|
|
|
|
elif ! nixfmt --check "$dest"; then
|
2024-07-27 13:08:22 +01:00
|
|
|
unformattedFiles+=("$dest")
|
2024-03-27 22:48:59 +00:00
|
|
|
fi
|
2024-07-23 20:03:15 +01:00
|
|
|
done < <(git diff -z --name-status ${{ env.baseRev }} -- '*.nix')
|
|
|
|
|
|
|
|
if (( "${#unformattedFiles[@]}" > 0 )); then
|
|
|
|
echo "Some new/changed Nix files are not properly formatted"
|
2024-06-26 00:08:17 +01:00
|
|
|
echo "Please run the following in \`nix-shell\`:"
|
2024-07-23 20:03:15 +01:00
|
|
|
echo "nixfmt ${unformattedFiles[*]@Q}"
|
2024-06-26 00:08:17 +01:00
|
|
|
exit 1
|
|
|
|
fi
|