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

Delete GetTestsProject task #47394

Merged
merged 1 commit into from
Mar 10, 2025
Merged

Conversation

Youssef1313
Copy link
Member

@mariam-abdulla This looks like a dead code?

@Youssef1313 Youssef1313 marked this pull request as ready for review March 9, 2025 12:41
@Copilot Copilot bot review requested due to automatic review settings March 9, 2025 12:41
Copy link

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@dotnet-issue-labeler dotnet-issue-labeler bot added the untriaged Request triage from a team member label Mar 9, 2025
Copy link

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

Choose a reason for hiding this comment

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

PR Overview

This PR removes the GetTestsProject file as it appears to be dead code.

  • Removed the entire GetTestsProject.cs file
  • Eliminated code that is no longer in use

Reviewed Changes

File Description
src/Tasks/Microsoft.NET.Build.Tasks/GetTestsProject.cs Deleted as part of dead code removal

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

Copy link
Member

@Forgind Forgind left a comment

Choose a reason for hiding this comment

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

It looks like this used to be used in the SDK and now isn't. If that was intentional, deleting it would be nice 🙂

@Youssef1313 Youssef1313 merged commit 7ef56b8 into main Mar 10, 2025
40 checks passed
@Youssef1313 Youssef1313 deleted the dev/ygerges/deadcode-gettestsproject branch March 10, 2025 08:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
untriaged Request triage from a team member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants