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

Unable to locate phpcs #48

Closed
spencerthayer opened this issue May 31, 2017 · 14 comments
Closed

Unable to locate phpcs #48

spencerthayer opened this issue May 31, 2017 · 14 comments
Assignees

Comments

@spencerthayer
Copy link

Issue #8 was closed and @hajekj reports that the issue is fixed for Windows. However, this is not the case for OS X. Please see comments below #8 (comment) .

@Budolfsen
Copy link

I can confirm that it is not working on OS X

@naknak987
Copy link

I can't get it working in windows though.

@msonowal
Copy link

me to macOS Sierra phpcs is installed globally via composer it terminal I can get output of phpcs i.e. PATH is present in environment but still now working

@auz1111
Copy link

auz1111 commented Oct 10, 2017

phpcs: Unable to locate phpcs. Please add phpcs to your global path or use composer depency manager to install it in your project locally

Still occurring on OS X.

@huabin
Copy link

huabin commented Oct 29, 2017

@auz1111 me too, phpcs: Unable to locate phpcs. Please add phpcs to your global path or use composer depency manager to install it in your project locally. still occurring on macOS.

OS: macOS High Serria
Version: 10.13.1 beta(17B46a)
VSCode version: 1.17.2

@ikappas
Copy link
Owner

ikappas commented Oct 29, 2017

@auz1111 Which version of the extension are you using? The marketplace one or the development one?

@rob-gordon
Copy link

Also having this problem on Sierra with the marketplace extension.

@ikappas
Copy link
Owner

ikappas commented Nov 1, 2017

I just pushed some major new features in develop branch that need testing and include:

ability to set phpcs.executablePath
ability to set phpcs.showWarnings
ability to set phpcs.showSources
ability to enable phpcs.trace.server with messages or verbose option for tracing errors.
ability to set phpcs.composerJsonPath to something other that workspaceRoot/composer.json
Your feedback is very welcome!

@ikappas ikappas self-assigned this Nov 2, 2017
@ikappas
Copy link
Owner

ikappas commented Nov 3, 2017

This issue should be resolved in latest develop branch so please consider testing your use cases.

@caramelomartins
Copy link

@ikappas I'm using Ubuntu 16.04 and I can confirm that I was able to go around this issue with version 1.0.0-beta.4. My use cases worked and I was able to see warnings from phpcsand fix files. I still had to set the full path to my phpcs global alias, was that what you intended?

@Wrenbjor
Copy link

Hello @ikappas, I am very new to using VSCode, are there instructions to installing/switching to the development branch of a plug-in so that I can test?

@caramelomartins
Copy link

@Wrenbjor Hey, you can see some instruction on running the development branch of vscode-phpcs here in the README of this repo.

@jeevad
Copy link

jeevad commented Feb 12, 2018

cmd run as admin and type following command

pear install PHP_CodeSniffer

@ikappas
Copy link
Owner

ikappas commented Feb 21, 2018

I just released vscode-phpcs 1.0.0 which supports the VSCode 1.20+ and includes many enhancements most notable of which are:

  1. multi root support
  2. lint as you type support
  3. automatic configuration search support
  4. ability to set phpcs.executablePath
  5. ability to set phpcs.showWarnings
  6. ability to set phpcs.showSources
  7. ability to set phpcs.composerJsonPath to something other that workspaceRoot/composer.json
  8. ability to enable phpcs.trace.server with messages or verbose option for tracing errors.

I would like to give my thanks to all the people that helped track down issues in the previous betas and hope the new features help you get things done easier.

On that note, I am closing this issue to properly track any new issues in the new release. If you find this issue persists or any other issue please feel free to open a new ticket for v.1.0.0.

@ikappas ikappas closed this as completed Feb 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests