composer-patches icon indicating copy to clipboard operation
composer-patches copied to clipboard

Fixes #178: Use dry-run to test patch to avoid leftover files/folders.

Open pescetti opened this issue 6 years ago • 2 comments

Fixes https://github.com/cweagans/composer-patches/issues/178 for branch 1.x.

pescetti avatar Mar 30 '18 18:03 pescetti

I've tested and confirmed that this fixes the problem of extraneous folders getting left around following composer updates, and it has no side effects / regressions. Is there anything holding up merging this?

Environments tested:

  • Ubuntu 18.04, Git 2.17.0, Composer 1.6.4
  • MacOS 10.13.4, Git 2.17.0, Composer 1.6.4
  • Debian 8.10, Git 2.1.4, Composer 1.6.3

danepowell avatar May 08 '18 18:05 danepowell

I also confirm that this fixes our problem (tested with Composer 2.0.6 on a large project with dozens of patches). Is there any chance that this will be merged?

prudloff-insite avatar Jun 07 '21 16:06 prudloff-insite

I've made a note to capture this in #447.

cweagans avatar Feb 04 '23 07:02 cweagans