Skip to content

Conversation

lowitea
Copy link
Contributor

@lowitea lowitea commented Dec 22, 2024

Hi! Thank you for this amazing tool!

I have written a new watcher for the NeoVim editor. I will be glad if you put him on the watchers list.

https://github.com/lowitea/aw-watcher.nvim


Important

Add NeoVim watcher aw-watcher.nvim to watchers.rst.

  • Documentation:
    • Adds NeoVim watcher aw-watcher.nvim by lowitea to the list of editor watchers in watchers.rst.

This description was created by Ellipsis for 3be809d. It will automatically update as commits are pushed.

Copy link

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Looks good to me! Reviewed everything up to 3be809d in 7 seconds

More details
  • Looked at 12 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 1 drafted comments based on config settings.
1. src/watchers.rst:47
  • Draft comment:
    Capitalize 'neovim' to 'Neovim' for consistency with other entries.
  • Reason this comment was not posted:
    Confidence changes required: 10%
    The PR adds a new watcher for NeoVim to the list of editor watchers. The format and placement are consistent with the existing entries. However, there is a minor inconsistency in capitalization for 'neovim'.

Workflow ID: wflow_nKxTxh1p6Z1deIa9


You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

@ErikBjare
Copy link
Member

Nice!

Curious what the differences are to the existing aw-watcher-vim plugin, maybe you can add a paragraph or two to the README? :)

@ErikBjare ErikBjare merged commit 3e6620f into ActivityWatch:master Dec 22, 2024
@lowitea
Copy link
Contributor Author

lowitea commented Dec 23, 2024

Ok! It can be useful. In short, aw-watcher-vim for Vim, but my plugin is for NeoVim, and is written completely in lua :) But I agree, it's worth writing about it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants