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

Drop MSTest doc that isn't VS specific #10617

Closed
wants to merge 1 commit into from

Conversation

Evangelink
Copy link
Contributor

Copy link
Contributor

@Evangelink : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Copy link
Contributor

Learn Build status updates of commit 513e39c:

✅ Validation status: passed

File Status Preview URL Details
docs/test/using-microsoft-visualstudio-testtools-unittesting-members-in-unit-tests.md ✅Succeeded n/a (file deleted or renamed)
docs/test/using-the-assert-classes.md ✅Succeeded n/a (file deleted or renamed)

For more details, please refer to the build report.

For any questions, please:

Copy link
Contributor

PRMerger Results

Issue Description
Deleted File(s) This PR deletes Markdown or YAML files owned by another author, or json file(s). The pull request must contain a comment from the pull request author confirming that all the file deletions are intentional before the pull request can be merged.
File Change Percent This PR contains file(s) with more than 30% file change.

@Evangelink
Copy link
Contributor Author

@Mikejo5000 I am not sure how to do some redirection to the new doc on dotnet side.

@Evangelink Evangelink marked this pull request as ready for review February 10, 2025 13:09
@Mikejo5000
Copy link
Collaborator

Mikejo5000 commented Feb 10, 2025

@Evangelink No problem. I will add that! Actually, this PR will need to be on the private repo because of deletions. I will open that this morning.

@Mikejo5000
Copy link
Collaborator

Mikejo5000 commented Feb 10, 2025

@Evangelink Amaury, can you let me know the PR that adds this content to .NET? (I took a quick look, didn't see it.) That PR will need to be merged before I can merge a PR for deletion and redirection of these files. Or, are these deleted files redundant and can be removed without adding new content to dotnet docs?

@Court72
Copy link
Contributor

Court72 commented Feb 10, 2025

@Evangelink No problem. I will add that! Actually, this PR will need to be on the private repo because of deletions. I will open that this morning.

@Mikejo5000 - I'll close this public PR for now. If you need it to be reopened, send an email to the
publicprs alias.

@Court72 Court72 closed this Feb 10, 2025
@Evangelink Evangelink deleted the drop-mstest-doc branch February 10, 2025 19:04
@Evangelink
Copy link
Contributor Author

@Evangelink Amaury, can you let me know the PR that adds this content to .NET? (I took a quick look, didn't see it.) That PR will need to be merged before I can merge a PR for deletion and redirection of these files. Or, are these deleted files redundant and can be removed without adding new content to dotnet docs?

The PR were merged some time ago, here are the pages:

@Mikejo5000
Copy link
Collaborator

Thanks! The new PR to delete the files is here:
https://github.com/MicrosoftDocs/visualstudio-docs-pr/pull/13602

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants