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

Improve php language config #31786

Merged
merged 3 commits into from
Jul 31, 2017
Merged

Conversation

jens1o
Copy link
Contributor

@jens1o jens1o commented Jul 31, 2017

Somehow the regex were invalid and couldn't match anything?

However, I fixed them. Taking a deeper look, they were quite good, only the else if case was missing.

For @rebornix

@msftclas
Copy link

@jens1o,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@jens1o
Copy link
Contributor Author

jens1o commented Jul 31, 2017

Feel free to modify it so it fits to your needs, as I'm going to be in bed in a few moments. 🛌

@rebornix
Copy link
Member

@jens1o thanks, it works great.

@rebornix rebornix merged commit d53301d into microsoft:master Jul 31, 2017
@jens1o jens1o deleted the jens1o-improve-php-cnfg branch August 1, 2017 10:01
@jens1o
Copy link
Contributor Author

jens1o commented Aug 1, 2017

No problem, happy to see this landing in 1.15. 😃

@github-actions github-actions bot locked and limited conversation to collaborators Mar 27, 2020
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.

3 participants