search

szeidler / composer-patches-cli

Adds a CLI wrapper for cweagans/composer-patches
MIT License
28 stars 8 forks source link

Evaluate to use a Trait or common ancestor for "patch-add" / "patch-remove" command which are very similar #13

Open szeidler opened 5 years ago

szeidler commented 5 years ago

Maybe this dialog (and following) should be put as a Trait so can be shared between the two patch-add and patch-remove classes? Or else have a common ancestor like PatchFileManipulatorAbstractCommand ?

Having a common ancestor could also help for adding/removing the patch with sharing all the options, because basically the two commands are very very similar.

Originally posted by @esolitos in https://github.com/szeidler/composer-patches-cli/pull/12