Skip to content
This repository has been archived by the owner on Jan 1, 2020. It is now read-only.

Use script for updating .gitignore #452

Conversation

weierophinney
Copy link
Member

Due to quoting issues between different operating systems, the script for removing the composer.lock entry from the .gitignore file was not working on Linux after a recent update to provide support for Windows. To provide a truly cross-platform way to work, this patch provides a script, bin/update-gitignore, which does the work. The composer.json entry is updated to call that script, which deletes itself on completion.

Fixes #450

@weierophinney weierophinney force-pushed the hotfix/450-cross-compat-gitignore branch 2 times, most recently from 9071546 to 497c816 Compare November 27, 2019 19:08
Copy link
Member

@michalbundyra michalbundyra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚢

Due to quoting issues between different operating systems, the script
for removing the `composer.lock` entry from the `.gitignore` file was
not working on Linux after a recent update to provide support for
Windows. To provide a truly cross-platform way to work, this patch
provides a script, `bin/update-gitignore`, which does the work. The
`composer.json` entry is updated to call that script, which deletes
itself on completion.
@weierophinney weierophinney force-pushed the hotfix/450-cross-compat-gitignore branch from 497c816 to edae8ac Compare November 27, 2019 19:11
weierophinney added a commit that referenced this pull request Nov 27, 2019
@weierophinney weierophinney merged commit edae8ac into zendframework:master Nov 27, 2019
@weierophinney weierophinney deleted the hotfix/450-cross-compat-gitignore branch November 27, 2019 19:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

composer scripts is not compatible with windows
2 participants