Conversation
- 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.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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>oto<leader>nfor semantic clarityTest plan