Skip to content

Added ability to specify editorPath using environment variable (#807) #856

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

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

Conversation

PawnTheAmazing
Copy link

@PawnTheAmazing PawnTheAmazing commented May 7, 2025

This is my first attempt at any extension coding, so feel free to give me tips if anything seems off.
But I noticed that someone had already reported this issue that I also wanted fixed, so it seemed like a good reason to dip my toes in.

Regex section is based on the setup in vscode-cpptools and seems to do the trick.

Only part I'm unsure about is the section in debugger_variables.test.ts. Was there a reason for it not calling clean_godot_path previously?

@PawnTheAmazing
Copy link
Author

Heya, just checking back in to make sure y'all aren't waiting for anything else from me to get this merged?
Totally fine if people are just busy. But haven't seen any updates for a bit, so wanted to make sure :)

@DaelonSuzuka
Copy link
Collaborator

No I've just been busy. I got a new job recently and I've been completely swamped.

@PawnTheAmazing
Copy link
Author

Aight, cool! No worries then, take your time! Just wanted to ensure I wasn't a bottleneck. Best of luck with the new job.

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