Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(config): override use cwd for root #86

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Ajaymamtora
Copy link

Description

The default behaviour of this plugin is not immediately obvious, it searches outside the CWD of nvim when searching for your local config file. I had a local config that defines LSP paths in the parent folder of the CWD for my work repos. This meant that when I opened a folder underneath this folder, neoconf would find the local config and the relative file paths I'd defined in my local config were invalid because they were relative to the parent dir.

For example, I'd defined angularls probe locations.

#61

It would be good to provide an option that stops this behaviour and limiting the search to the cwd.

Ive been using code in my fork for a few weeks and have had no issues.

Related Issue(s)

This fixes #61

Screenshots

@Ajaymamtora Ajaymamtora changed the title override use cwd for root feat(config) override use cwd for root Aug 24, 2024
@Ajaymamtora Ajaymamtora changed the title feat(config) override use cwd for root feat(config): override use cwd for root Aug 24, 2024
Copy link
Contributor

This PR is stale because it has been open 30 days with no activity.

@github-actions github-actions bot added the stale label Sep 24, 2024
@Ajaymamtora
Copy link
Author

.

@github-actions github-actions bot removed the stale label Sep 25, 2024
Copy link
Contributor

This PR is stale because it has been open 30 days with no activity.

@github-actions github-actions bot added the stale label Oct 27, 2024
@Ajaymamtora
Copy link
Author

.

@github-actions github-actions bot removed the stale label Oct 29, 2024
Copy link
Contributor

This PR is stale because it has been open 30 days with no activity.

@github-actions github-actions bot added the stale label Nov 28, 2024
@Ajaymamtora
Copy link
Author

.

@github-actions github-actions bot removed the stale label Nov 29, 2024
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.

bug: State not cleared when changing cwd/workspaces
1 participant