Show parent directory with filename in lualine statusline#163
Conversation
- Move sections config out of options where it was incorrectly nested - Add lualine_c with filename path=4 to display parent_dir/filename Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Code ReviewSummaryThis PR fixes a configuration bug in the lualine statusline plugin and adds parent directory display to the filename component. The changes are well-focused and address a real issue. ✅ Positive AspectsBug Fix: The primary change correctly moves the Improved Navigation Context: Adding Clean Code Structure: The indentation and formatting are consistent with Lua conventions. 🔍 Observations & ConsiderationsConfiguration Validity: The Empty Section: Setting Other Sections: The PR only configures
If you want a more minimal statusline, you might consider explicitly configuring the other sections as well. 📋 Test PlanThe test plan in the PR description is appropriate. Consider also testing:
RecommendationLGTM - The changes are technically correct and improve the configuration. The bug fix is important and the feature addition is useful. No blocking issues identified. The only question is whether hiding the mode indicator ( Review generated by Claude Code |
Summary
sectionsconfig that was incorrectly nested insideoptions(had no effect)lualine_cwithfilenamecomponent usingpath = 4to displayparent_dir/filenameformatTest plan
parent_dir/filename(e.g.plugins/lualine.lua)🤖 Generated with Claude Code