Skip to content

fix: don't duplicate --allow-all for deno debug configuration#2308

Merged
connor4312 merged 2 commits into
microsoft:mainfrom
nayeemrmn:deno-debug-allow-all-check
Jan 12, 2026
Merged

fix: don't duplicate --allow-all for deno debug configuration#2308
connor4312 merged 2 commits into
microsoft:mainfrom
nayeemrmn:deno-debug-allow-all-check

Conversation

@nayeemrmn

@nayeemrmn nayeemrmn commented Jan 12, 2026

Copy link
Copy Markdown
Contributor

Doesn't append --allow-all if it's specified by the user

@nayeemrmn

Copy link
Copy Markdown
Contributor Author

@microsoft-github-policy-service agree [company="denoland"]

@nayeemrmn

Copy link
Copy Markdown
Contributor Author

@microsoft-github-policy-service agree company="denoland"

Comment thread src/ui/configuration/nodeDebugConfigurationResolver.ts

@connor4312 connor4312 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks!

@connor4312 connor4312 enabled auto-merge (squash) January 12, 2026 18:35
@vs-code-engineering vs-code-engineering Bot added this to the January 2026 milestone Jan 12, 2026
@connor4312 connor4312 merged commit ef7f6e7 into microsoft:main Jan 12, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants