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

Fix powershell #1

Merged
merged 4 commits into from
Aug 1, 2024
Merged

Fix powershell #1

merged 4 commits into from
Aug 1, 2024

Conversation

TheLeoP
Copy link
Contributor

@TheLeoP TheLeoP commented Aug 1, 2024

Fixes the problems stated in TheLeoP/powershell.nvim#3

Also,

  • I couldn't execute your config locally because you weren't calling require('f0oster.packer'), so I fix it
  • I updated the listeners for dap-ui (maybe you weren't using the close listeners on purpose(?), you could delete them if that's the case)
  • I removed dap-powershell because powershell.nvim already handles debugging and both configurations may clash (or your would only use one of them). dap-powershell does support interactive debugging (through the dap-repl), but it doesn't work on windows (because nvim-dap does not handle named pipes nicely on Windows). powershell.nvim works on Windows (including debugging), but it does not support interactive debugging because of a bug with Powershell Editor Services

@TheLeoP
Copy link
Contributor Author

TheLeoP commented Aug 1, 2024

Additionally, I haven't use packer since it was archived, so I don't remember exactly how to use it. You may need to :PackerCompile or :PackerSync (or both?) after merging this changes

@f0oster
Copy link
Owner

f0oster commented Aug 1, 2024

@TheLeoP Thanks a lot for your help, I really appreciate it - all seems to be working now. I'm pretty new to neovim and the ecosystem/config is a little confusing so far.

  • I couldn't execute your config locally because you weren't calling require('f0oster.packer'), so I fix it

I was typically sourcing packer.lua to run Packer commands, but this change is fine.

  • I updated the listeners for dap-ui (maybe you weren't using the close listeners on purpose(?), you could delete them if that's the case)

Thanks for fixing that. I hadn't intentionally configured dap-ui that way - I was just messing about with config last night.

  • I removed dap-powershell because powershell.nvim already handles debugging and both configurations may clash (or your would only use one of them). dap-powershell does support interactive debugging (through the dap-repl), but it doesn't work on windows (because nvim-dap does not handle named pipes nicely on Windows). powershell.nvim works on Windows (including debugging), but it does not support interactive debugging because of a bug with Powershell Editor Services

Thanks - I did see this issue last night. I do plan to set this config up on Windows as well, so it's good to know.

Additionally, I haven't use packer since it was archived, so I don't remember exactly how to use it. You may need to :PackerCompile or :PackerSync (or both?) after merging this changes

Yep, I ran a :PackerSync and all is good. Thanks again! You've been a massive help. I'll need to spend a weekend looking at migrating away from Packer and also generally updating my set up some time soon I think.

@f0oster f0oster merged commit 3684cf9 into f0oster:main Aug 1, 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.

2 participants