Skip to content

fix: commands setup#8

Merged
hjetmundsen merged 1 commit into
mainfrom
kchen6/fix-config
Aug 18, 2025
Merged

fix: commands setup#8
hjetmundsen merged 1 commit into
mainfrom
kchen6/fix-config

Conversation

@kunchenguid

Copy link
Copy Markdown
Contributor

Pull Request

Description

Fixing a few things -

  1. no need to set config with lazy.nvim - setup() will be called automatically
  2. should probably not recommend the cmd trigger, because it means setup() will not be called until people run "RovoDevToggle" manually
  3. commands setup step is fixed

Type of Change

Please check the options that are relevant:

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Code refactoring
  • Performance improvement
  • Other (please describe):

Checklist

Please check all that apply:

  • I have read the CONTRIBUTING document
  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have tested with the actual Rovo Dev CLI tool
  • I have tested in different environments (if applicable)

Screenshots (if applicable)

Add screenshots to help explain your changes if they include visual elements.

Additional Notes

Add any other context about the PR here.

Comment thread lua/rovo-dev/init.lua

-- Register commands
commands.register_commands(M.config)
commands.register_commands(M)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

the toggle method is on M instead of M.config

@hjetmundsen hjetmundsen merged commit 779110e into main Aug 18, 2025
2 checks passed
@kunchenguid kunchenguid deleted the kchen6/fix-config branch August 18, 2025 17:06
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