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

Reflactor #12

Closed
14 tasks done
jinzhongjia opened this issue Aug 4, 2023 · 3 comments
Closed
14 tasks done

Reflactor #12

jinzhongjia opened this issue Aug 4, 2023 · 3 comments
Assignees
Labels
enhancement New feature or request

Comments

@jinzhongjia
Copy link
Owner

jinzhongjia commented Aug 4, 2023

I created this plugin just because I needed a stable lsp plugin (didn't follow many specifications), now I want to refactor the logic of the entire code and add new features

Some real-world testing is required before merging

Here are the features I think need to be added:

If you can see this issue, you can discuss what features need to be added!!

Features that have been refactored:

note:

The following are all completed

code action should provide registration method, for lua's code action
diagnostic should provide a diagnostic overview of the current workspace and files

Problems:

the keymap only will remove, we should restore keymap

@jinzhongjia jinzhongjia pinned this issue Aug 4, 2023
@jinzhongjia
Copy link
Owner Author

jinzhongjia commented Aug 5, 2023

Development specification:

  1. Each module contains its own init function
  2. Create command when module init
  3. Each module is decoupled, can be referenced and used separately
  4. Each module has func run
  5. As neovim is the host system, its plugins should be kept as minimally intrusive as possible

@jinzhongjia jinzhongjia added the enhancement New feature or request label Aug 10, 2023
@jinzhongjia
Copy link
Owner Author

In feature, maybe some nvim-builtin func need to be rewrite ?
such as get next diagnostic, it do repeated operations

@jinzhongjia jinzhongjia self-assigned this Aug 19, 2023
@jinzhongjia
Copy link
Owner Author

now, I didn't know how to display callHierarchy

@jinzhongjia jinzhongjia unpinned this issue Nov 17, 2023
@jinzhongjia jinzhongjia mentioned this issue Nov 17, 2023
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

When branches are created from issues, their pull requests are automatically linked.

1 participant