Skip to content

Change mini.files function from 'later' to 'now'#37

Closed
wdfry wants to merge 2 commits intonvim-mini:mainfrom
wdfry:mini.files-later-to-now
Closed

Change mini.files function from 'later' to 'now'#37
wdfry wants to merge 2 commits intonvim-mini:mainfrom
wdfry:mini.files-later-to-now

Conversation

@wdfry
Copy link
Contributor

@wdfry wdfry commented Feb 16, 2026

  • This PR does not suggest changes based on personal taste (enabling new or changing value of existing option, install new plugin)

Changed the mini.files plugin to use the 'now' function instead of 'later' so that netrw doesn't fire before plugin when launching nvim with 'nvim .'

@echasnovski
Copy link
Member

Thanks for the PR!

Adjusting 'mini.files' to handle the nvim . case indeed makes sense. Shame on me, though, as couple of weeks ago I specifically looked through all 'mini.nvim' modules to see if they may be needed during startup. Failed to detect the default explorer case :(

I'd ask to use now_if_args(), as this is more performant for the "default" case of starting with just nvim.

@wdfry
Copy link
Contributor Author

wdfry commented Feb 16, 2026

Yep, that makes sense. I've adjusted it.

@echasnovski
Copy link
Member

Thanks! It required a bit more work with shuffling around into different "section", so I did it in the separate commit (with you as co-author). Should now be resolved on the latest main.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants