Skip to content

Conversation

@paris3200
Copy link
Owner

Summary

• Enhanced rename function with fzf-lua file preview for better user experience
• Added note rename functionality with automatic link updates
• Comprehensive README and documentation overhaul with better examples
BREAKING: Changed default keybinding prefix from <leader>o to <leader>n for semantic clarity

Test plan

  • Rename functionality tested with link updates
  • File preview working in fzf-lua interface
  • Documentation updated and help tags regenerated
  • All keybindings updated to new prefix

paris3200 added 7 commits July 6, 2025 10:55
- Update test workflow to run on push and PR events for develop branch
- Ensures CI/CD runs on feature accumulation branch before merging to main
- Add M.rename_note() function that finds and updates all link references
- Support both [[note]] and [[note|display]] link formats
- Add :MarkdownNotesRename command and <leader>or keybinding
- Include comprehensive input validation and safety checks
- Add extensive test suite covering edge cases
- Update documentation with examples and usage
…19)

BREAKING CHANGE: All default keybindings now use <leader>n prefix instead of <leader>o for better semantic meaning as a notes plugin.

Changes:
- Update all default keybindings in config.lua
- Update documentation in README.md and help files
- Users relying on default keybindings must update their muscle memory or explicitly configure <leader>o mappings

Migration: Users who prefer <leader>o can restore it by setting custom mappings in their configuration.
- Redesign README with modern structure, badges, and clear navigation
- Add Quick Start section for immediate productivity
- Reorganize content flow from basic to advanced usage
- Update all default paths from ~/repos/notes to ~/notes
- Add practical examples and real-world templates
- Improve keybinding reference with grouped categories
- Enhance troubleshooting section with common solutions
- Remove Unicode/emoji characters from Vim help documentation
- Convert markdown tables to proper Vim help formatting
- Update LICENSE copyright to Jason Paris
- Remove marksman LSP suggestion to simplify requirements
- Add comprehensive workspace documentation with examples
- Add config test for merging user template variables with defaults
- Add integration test for config-defined template variables in substitution
- Add test for custom variable override behavior
- Verify both function and static value template variables work correctly
* feat: add luacheck linting with configuration

- Add .luacheckrc with Neovim-specific linting rules
- Add Makefile with lint, test, and utility targets
- Configure appropriate warning ignores for plugin development
- Set 100 character line limit for consistent formatting

* fix: resolve all luacheck linting warnings and improve code quality

- Fix logic issues by replacing inefficient loops with next() function calls
- Break long lines to comply with 100 character limit
- Remove trailing whitespace throughout codebase
- Fix read-only field warnings in tests using proper metatable approach
- Add test-setup.sh script for isolated testing environment
- Achieve zero linting warnings across all Lua files

* ci: add linting to GitHub Actions workflow and update contributing docs

- Rename workflow from 'Tests' to 'CI' for broader scope
- Add parallel lint job with luacheck installation
- Update test job to use make commands for consistency
- Enhance README contributing section with development tools documentation
- Document all Makefile targets and CI integration
- Add fzf-lua preview interface showing files that will be updated during rename
- New ui.show_rename_preview config option (default: true) to control behavior
- Maintain backward compatibility with skip_ui option for tests
- Fix buffer management issue where both old and new file buffers remained open
- Remove redundant confirmation dialog after preview for better UX
- Update all documentation (README, help files) with new functionality
- Add comprehensive test coverage for config option and buffer handling
- Clean up debug statements and fix linting issues
- Improve luacheck configuration for better code formatting

Users can now see exactly which files will be affected before confirming
a rename operation, with proper buffer management and the ability to preview
file contents. The preview can be disabled if desired.
@paris3200 paris3200 merged commit 8a1eb4f into main Jul 7, 2025
8 checks passed
@paris3200 paris3200 deleted the develop branch July 7, 2025 15:33
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