pretty-quick: pretty-quick does not find any changed file

I’m working with node 12.18.1, prettier@2.0.5, and pretty-quick@3.1.0 on a windows 10 computer. I have this strange behaviour:

$ git status
On branch feature/code-quality
Your branch is up to date with 'origin/feature/code-quality'.
Changes to be committed:
  (use "git reset HEAD <file>..." to unstage)
        modified:   src/app/app.component.spec.ts

then

$ npx pretty-quick --staged
�  Finding changed files since git revision 2ba99b6.
�  Found 0 changed files.
✅  Everything is awesome!

but

$ npx prettier --check **
Checking formatting...
src\app\app.component.spec.ts
Code style issues found in the above file(s). Forgot to run Prettier?

Any idea ?

About this issue

Original URL
State: open
Created 3 years ago
Comments: 15 (1 by maintainers)

Most upvoted comments

i was also getting the git revision null message when i ran npx pretty-quick --staged

Needed to add the --branch main flag since it compares against master by default (as it says in the docs - which i did not read).

+11

adanilev on Mar 30, 2022

I have a similar problem but it just returns “Git Revision null”

TomCockram on Apr 8, 2021

@TomCockram same same. What’s the environment where you get that null? Also in a git action?

$ npx pretty-quick --check --branch develop && ng lint
🔍  Finding changed files since git revision null.
Command failed with exit code 2 (ENOENT): git diff --name-only --diff-filter=ACMRTUB
spawnSync git ENOENT
error Command failed with exit code 2.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

I get this behaviour when performing this in a GitHub action.

TomCockram on May 13, 2021

How to use this in GitHub Actions with actions/checkout@v3? I have tried to set fetch-depth=0 but still no changes found. Locally the same command works nicely so I guess the problem is in Github’s way to do checkout.

heikkis on Aug 18, 2023

i was also getting the git revision null message when i ran npx pretty-quick --staged

Needed to add the --branch main flag since it compares against master by default (as it says in the docs - which i did not read).

I had an issue where pretty-quick was including files in our monorepo in parent folders relative to where I was running it, but I hadn’t changed. Turns out using this flag against the target branch fixed my issue. Thanks!

andymerskin on Apr 12, 2023

The issue is, that pretty-quick uses git under the hood. If you use alpine or ubuntu slim images, they usually don’t have git command out of the box. Make sure, that you install git in your CI runner.

zoltan-nz on Mar 5, 2022