zerol

Results 36 comments of zerol

I am planning to use the [copilot-node-server](https://www.npmjs.com/package/copilot-node-server) NPM package (which is used by https://github.com/TerminalFi/LSP-copilot). The primary tasks that must be completed are: - Installing the NPM package on behalf of...

There are a few potential issues to consider: - `npm` becomes a new dependency. - Some users may not want to install the npm package globally (though it is the...

> Node is already a dependency, which always embeds npm, AFAIK. On Ubuntu and Arch, `npm` is not automatically installed when installing `nodejs`, as it's an optional dependency. But yes,...

@artlogic (The item was crossed out in the readme a long while ago.) I have no plans to add the package to MELPA. However, I welcome any contributions.

@sshaw @jgarte Yes, it makes sense but I need help. A few things to consider: - Store target commit ID of copilot.vim in a variable. - Prompt and auto-install binary...

@emil-vdw Sorry for not making it clear. The problem is, before submitting this plugin to MELPA, we may need to get rid of the `dist` folder to make it a...

> The [tokei](https://github.com/XAMPPRocky/tokei) program is referenced below in the `defcustom` form but the tokei.el package does not try to install the package for the user: > > https://github.com/nagy/tokei.el/blob/master/tokei.el#L47 > >...

We need to follow the instructions here: https://github.com/melpa/melpa/blob/master/CONTRIBUTING.org 1. clean up unnecessary files (dist folder, GitHub workflow) 2. add a new license (I suggest MIT, which is GPL-compatible) 3. update...

You could add a hook to `copilot--display-overlay-completion`. If the argument `completion ` contains anything bad, skip the function body.

The rolling window has already been implemented in copilot.el. However, it may require rework for the compatibility with the LSP protocol, such as `textDocument/didChange` messages.