bmewburn / vscode-intelephense

PHP intellisense for Visual Studio Code
https://intelephense.com
Other
1.6k stars 94 forks source link

GitHub Copilot autocommit dockblock #2257

Open exbit opened 2 years ago

exbit commented 2 years ago

If you install GitHub Copilot, then dockblock autocommit does not work:

When I type: /**

instead of autocomplete intelephense, autocomplete copilot appears:

snap_screen_20220710124332

If copilot is disabled, then intelephense autocomplete works:

/**
 * @param \App\Models\Transaction $transaction
 *
 * @return void
 *
 * @throws \Illuminate\Contracts\Container\BindingResolutionException
 * @throws \GuzzleHttp\Exception\GuzzleException
 * @throws \Throwable
 */
public function __invoke(Transaction $transaction): void
{

Debian 11, PHP 8.1, PHP Intelephense v1.8.2

bradietilley commented 2 years ago

Yep, I get this a bit. Also, namespace imports aren't suggested half the time (half the time they are?).

$product = Product should yield an autocomplete list including an import suggestion of App\Models\Product but copilot thinks its clever and suggests code without the import (e.g. Product::first()). Even if you tab in the copilot suggestion and move the cursor back to the class name (and backspace then retype last character) it won't suggest the class import.

The two plugins are clashing.. out of Intelephense and copilot, it'll be copilot that's first to go for me, but I'd like to see if they can work harmoniously

gautiermichelin commented 1 year ago

+1