-
-
Notifications
You must be signed in to change notification settings - Fork 309
feat(test): add crazy-coverage-nvim plugin #1724
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
Conversation
- Add new coverage category - Add crazy-coverage.nvim plugin for displaying code coverage overlays
Review ChecklistDoes this PR follow the [Contribution Guidelines](development guidelines)? Following is a partial checklist: Proper conventional commit scoping:
|
Uzaaft
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wrong category. Move it to test category. And cleanup your code and the comments. Most of them arent needed.
The config call can me removed.
If the config you are passing is the default settings, just set it as an empty table.
|
Also tidy up your readme.md. Look at other readme's for inspiration. 1 line should be enough for the desc of the plugin. |
|
@Uzaaft I'll cleanup my code and readme. I have a suggestion of creating new section for coverage. I'm currently in discussion(#2090) with the maintainers of |
|
No its not. Its based off https://github.com/neurosnap/neovimcraft |
|
Ok but neovimcraft scraps awesome-neovim for tags. |
|
Not gonna have a fight with you over this. I cleaned up the useless comments and moved it. Merging. |
📑 Description
📖 Additional Information