-
Notifications
You must be signed in to change notification settings - Fork 4
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
Version v0.1 #1
Comments
/bounty 4000 |
|
@olegklimov Would love to work on this. |
Please assign to me. |
I'd love to work on this @olegklimov could you assign it to me? |
@olegklimov I would also love to work on this one |
/attempt 1 |
@yuvrajjsingh0: The Refact.ai team prefers to assign a single contributor to the issue rather than let anyone attempt it right away. We recommend waiting for a confirmation from a member before getting started. |
@olegklimov can you elaborate some more upon how you need the status bar to be? |
Hi everyone. I'll assign this to James because he's good, we worked with him before. |
@olegklimov No worries. (The question on my mind and I'm sure on others' is: how can we show you if we are any good if we are never given chance when new tasks are available? Perhaps you have a few low-ticket tasks that you are planning to throw open to everyone? You don't have to answer but it would nice if you did.) |
/attempt 1 |
@PlanetKumbhaj: Another person is already attempting this issue. Please don't start working on this issue unless you were explicitly asked to do so. |
any updates on this? |
💡 @digital-phoenix submitted a pull request that claims the bounty. You can visit your bounty board to reward. |
@digital-phoenix has a first version, but no one checked it yet, because we got stuck with the VS Classic plugin. We'll try to unstuck it as soon as possible. |
Hey, I would like to work on this project. |
🎉🎈 @digital-phoenix has been awarded $4,000! 🎈🎊 |
To everybody who's tracking this implementation -- it works, there are a couple of bugs (like you need to restart IDE for a change in settings to apply) but it works, we'll fix it. We'll post it into the marketplace so it's easier to install. But you can install from source already. |
Following https://github.com/smallcloudai/refact-vs-classic/ , we need a first version of a plugin for Sublime Text that starts refact-lsp server, has completion, status bar, settings, and that's about it.
Completion should behave generally like in VS Code, more details about multiline and single line completion are here in a VS Classic issue and here is some more.
Clarification: this has nothing to do with completions that go inside a popup. Here we are only interested in inline completions (multiline or single line), also known as "grey text" or "ghost text".
Refact-LSP is an executable file that gets built here, download one for your platform, run it with --help to see what kind of options it accepts.
The relevant LSP part is textDocument/didOpen, textDocument/didChange, textDocument/didClose, workspace/didDeleteFiles -- to inform the LSP server about the source files open or changed. This is used for context, for a good completion for file1 probably it needs a context from file2. That's why refact-lsp needs those calls to happen.
The completion itself is a LSP call
refact/getCompletions
, but there's an effort to switch to a more standard 3.18.Status bar like in this VS issue needs an idle state, rotating icon when it asks for a new completion, and an error state that shows text error in a tooltip.
Settings can be similar to VS Code settings, those are mainly go directly to refact-lsp command line. One that doesn't is "pause", it should be accessible via a context menu in the status bar or a hotkey -- an ability to pause completions if they get annoying in some cases.
This bounty supersedes the previous attempt over here that got stalled.
The text was updated successfully, but these errors were encountered: