Skip to content

fix(config): handle Windows rnvim config path#599

Merged
jalvesaq merged 1 commit into
mainfrom
codex/fix-windows-rnvim-config-path
Jun 27, 2026
Merged

fix(config): handle Windows rnvim config path#599
jalvesaq merged 1 commit into
mainfrom
codex/fix-windows-rnvim-config-path

Conversation

@hongyuanjia

@hongyuanjia hongyuanjia commented Jun 27, 2026

Copy link
Copy Markdown
Contributor

Summary

Fix the per-buffer rnvim_config.lua lookup so it works with Windows buffer paths. The previous slash-only Lua pattern could return nil for backslash-separated paths and fail before checking whether the config file exists.

This bug was introduced in PR #515.

Changes

  • Build the config path with vim.fs.dirname() and vim.fs.joinpath() instead of a hard-coded / pattern.
  • Guard the lookup when the current buffer has no file name.

@hongyuanjia hongyuanjia marked this pull request as ready for review June 27, 2026 03:17
@jalvesaq

Copy link
Copy Markdown
Member

Thank you!

@jalvesaq jalvesaq merged commit f0b8ff4 into main Jun 27, 2026
13 checks passed
@jalvesaq jalvesaq deleted the codex/fix-windows-rnvim-config-path branch June 27, 2026 10:31
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