Skip to content

remove stale code for displaying config error #25269

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Jul 10, 2025
Merged

remove stale code for displaying config error #25269

merged 3 commits into from
Jul 10, 2025

Conversation

eleanorjboyd
Copy link
Member

@eleanorjboyd eleanorjboyd commented Jul 10, 2025 •
edited
Loading

Two parts:

  • removes notification on run without configuration, this was meant to be removed as stated in the TODO comment
  • removes unused function displayTestFrameworkError & tests related to it

@eleanorjboyd eleanorjboyd self-assigned this Jul 10, 2025
@eleanorjboyd eleanorjboyd added debt Covers everything internal: CI, testing, refactoring of the codebase, etc. skip-issue-check labels Jul 10, 2025
@vs-code-engineering VS Code Engineering
Copy link

⚠️ 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 July 2025 milestone Jul 10, 2025
@eleanorjboyd eleanorjboyd enabled auto-merge (squash) July 10, 2025 15:45
@eleanorjboyd eleanorjboyd merged commit 8810766 into microsoft:main Jul 10, 2025
46 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lramos15 lramos15 lramos15 approved these changes

Assignees

@eleanorjboyd eleanorjboyd

Labels
debt Covers everything internal: CI, testing, refactoring of the codebase, etc. skip-issue-check
Projects
None yet
Milestone
July 2025
Development

Successfully merging this pull request may close these issues.
2 participants
@eleanorjboyd @lramos15