Open staabm opened 5 years ago
The merge driver itself would be inappropriate for this purpose, since it has already been fully executed by the time the user is manually fixing merge conflicts and where this edge case would occur. Instead, this would be best performed in a pre-commit hook or some other appropriate git hook. I think it would be in the scope of this project, if you or anyone else would like to propose a sample git hook to include in the repo which can be manually installed just like the driver itself.
The readme mentions
Would it be possible that the merge driver automatically invokes
composer validate
after a merge to make sure (and the user aware) if the generated files contain a syntax error after merge (so we dont need to manually be aware of this edge case)?