ComposerLintReview Error about static-review HOT 6 CLOSED

static-review doesn't include composer.lock in the repository, so this should be solved by running composer update on your local machine. If that doesn't work, try rm -rf vendor/sjparkinson/static-review followed by composer install.

from static-review.

The composer.json file is not part of static-review in any way. Our project uses static-review as a git hook, and one of the reviews is ComposerLintReview which fails when we try to commit changes to our composer.json.

from static-review.

Well then it looks like you tried to commit composer.json without composer.lock being in sync.

from static-review.

Our project uses static-review as a git hook, and one of the reviews is ComposerLintReview which fails when we try to commit changes to our composer.json.

So static review creates a temporary version of the staged file that you want to commit and reviews that.

Have a go running git show :composer.json and ensure that the output is a valid file. Feel free to include it here.

from static-review.

I figured it out.

The cached composer.json file static-review uses checks if it's out-of-sync against any cached composer.lock, meaning that if I only commit the composer.json file it'll check out-of-sync against an old composer.lock (as long as there is one in the cache).

In my case we once upon a time had composer.lock in the repo but later decided to remove it (due to the repo being a library). But I still have an old copy of it in static-review's cache, so every time I try to commit changes to composer.json it'll complain.

I removed the cached composer.lock and now I can successfully commit composer.json. Woo!

from static-review.

In my case we once upon a time had composer.lock in the repo but later decided to remove it (due to the repo being a library). But I still have an old copy of it in static-review's cache, so every time I try to commit changes to composer.json it'll complain.

Ah yes, I haven't done much work on cleaning up the temporary files, nice work figuring it out! I'll open a new issue to look at cleaning up the temporary files that the git version control class creates.

from static-review.