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

Require path in phpcbf #2141

Merged
merged 2 commits into from
May 28, 2018
Merged

Require path in phpcbf #2141

merged 2 commits into from
May 28, 2018

Conversation

stevenzeck
Copy link
Contributor

@stevenzeck stevenzeck commented May 22, 2018

What does this implement/fix? Explain your changes.

Requires the path library in phpcbf.coffee as path is being used but isn't required, thus causing an error. Also adds the optional flag to PHPCBF executable.
...

Does this close any currently open issues?

Closes #2140
...

Any other comments?

...

Checklist

Check all those that are applicable and complete.

  • Merged with latest master branch
  • Regenerate documentation with npm run docs
  • Add change details to CHANGELOG.md under "Next" section
  • Added examples for testing to examples/ directory
  • Travis CI passes (Mac support)
  • AppVeyor passes (Windows support)

@stevenzeck stevenzeck requested a review from Glavin001 May 23, 2018 15:18
@stevenzeck
Copy link
Contributor Author

@Glavin001 please give this a quick look and I'll merge.

@stevenzeck stevenzeck self-assigned this May 23, 2018
@stevenzeck stevenzeck added the bug label May 23, 2018
@Glavin001
Copy link
Owner

@szeck87 Need a Changleog entry for this before publishing. Feel free to merge and publish when you're ready 👍

@stevenzeck stevenzeck merged commit 7f9d838 into master May 28, 2018
@stevenzeck stevenzeck deleted the phpcbf-patch1 branch May 28, 2018 22:14
@stevenzeck stevenzeck added this to the Next Release milestone May 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Phpcbf path is not defined
2 participants