Skip to content

fix: use wrapper functions for easier testing - #24941

Merged
Karthik Nadig (karthiknadig) merged 1 commit into
microsoft:mainfrom
karthiknadig:issue-24426
Apr 2, 2025
Merged

fix: use wrapper functions for easier testing#24941
Karthik Nadig (karthiknadig) merged 1 commit into
microsoft:mainfrom
karthiknadig:issue-24426

Conversation

@karthiknadig

Copy link
Copy Markdown
Member

Fixes #24426

@karthiknadig Karthik Nadig (karthiknadig) added the debt Code quality issues label Mar 31, 2025
@karthiknadig Karthik Nadig (karthiknadig) added the skip tests Updates to tests unnecessary label Apr 2, 2025
@karthiknadig
Karthik Nadig (karthiknadig) marked this pull request as ready for review April 2, 2025 00:24
@vs-code-engineering

Copy link
Copy Markdown

⚠️ This PR originates from a fork. Due to security restrictions, pipelines from forks are no longer triggered automatically. Learn more.

If the changes appear safe, you can manually trigger the pipeline by commenting /AzurePipelines run.

@vs-code-engineering vs-code-engineering Bot added this to the April 2025 milestone Apr 2, 2025
@karthiknadig
Karthik Nadig (karthiknadig) merged commit 6a20c9c into microsoft:main Apr 2, 2025
@karthiknadig
Karthik Nadig (karthiknadig) deleted the issue-24426 branch April 2, 2025 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

debt Code quality issues skip tests Updates to tests unnecessary

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Refactor Native REPL code to avoid using wide scope VS Code namespace

2 participants