Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support ci:php:lint on Windows #740

Closed
JakeQZ opened this issue Sep 29, 2019 · 3 comments
Closed

Support ci:php:lint on Windows #740

JakeQZ opened this issue Sep 29, 2019 · 3 comments
Assignees
Milestone

Comments

@JakeQZ
Copy link
Contributor

JakeQZ commented Sep 29, 2019

The Composer script for php -l uses a *nix-specific command and does not work on Windows.

We should find a way of allowing developers to run the whole of the ci script on whatever platform they're using. (I'm guessing it works OK on macOS as that's Unix-based. None of the other parts of the ci script have an issue on Windows.)

@JakeQZ JakeQZ added this to the 4.0.0 milestone Sep 29, 2019
@SignpostMarv
Copy link
Contributor

appears to work under gitbash.

@oliverklee
Copy link
Contributor

@SignpostMarv Just wanted to let you know that I greatly appreciate you having started to contribute to Emogrifier with pull requests and ticket comments. ❤️ Keep 'em coming! 🍪

@JakeQZ
Copy link
Contributor Author

JakeQZ commented Oct 1, 2019

appears to work under gitbash.

I'm just using the command prompt in Windows, where all other commands work fine. I could start using gitbash. But I think this extra requirement would discourage people from contributing, so I'd prefer to have something that will work on all environments.

SignpostMarv added a commit to SignpostMarv/emogrifier that referenced this issue Oct 1, 2019
@JakeQZ JakeQZ closed this as completed in 926e421 Oct 1, 2019
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants