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

Prevent corrupted breakpoints due to invalid sourceDirs, add more logging #189

Merged
merged 2 commits into from
May 22, 2024

Conversation

TwitchBronBron
Copy link
Member

@TwitchBronBron TwitchBronBron commented May 17, 2024

Prevent breakpoints from becoming corrupted due to bad sourceDirs entries.
Also adds many .debug logs during the breakpoint flows.

@TwitchBronBron TwitchBronBron added the create-vsix PRs with this tag will trigger a vsix build on vscode-brightscript-language for every push label May 17, 2024
@rokucommunity-bot
Copy link

Hey there! I just built a new version of the vscode extension based on 2e76ff1. You can download the .vsix here and then follow these installation instructions.

@TwitchBronBron TwitchBronBron changed the title Add more logging around breakpoints Don't allow bad sourceDirs to corrupt breakpoints, add more logging May 22, 2024
@TwitchBronBron TwitchBronBron changed the title Don't allow bad sourceDirs to corrupt breakpoints, add more logging Prevent corrupted breakpoints due to invalid sourceDirs, add more logging May 22, 2024
@TwitchBronBron TwitchBronBron removed the create-vsix PRs with this tag will trigger a vsix build on vscode-brightscript-language for every push label May 22, 2024
@rokucommunity-bot
Copy link

Hey there! I just built a new version of the vscode extension based on b6cc37a. You can download the .vsix here and then follow these installation instructions.

@TwitchBronBron TwitchBronBron merged commit 2de201d into master May 22, 2024
8 checks passed
@TwitchBronBron TwitchBronBron deleted the better-logging branch May 22, 2024 13:42
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.

1 participant