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

Added article Generate Unit Tests with Copilot #44228

Merged
merged 22 commits into from
Mar 20, 2025

Conversation

sigmade
Copy link
Contributor

@sigmade sigmade commented Jan 12, 2025

Summary

Added article about Generate Unit Tests with Copilot and Visual Studio


Internal previews

📄 File 🔗 Preview link
docs/core/testing/unit-testing-with-copilot.md Generate unit tests with GitHub Copilot
docs/navigate/devops-testing/toc.yml docs/navigate/devops-testing/toc

@sigmade sigmade requested review from IEvangelist and a team as code owners January 12, 2025 10:20
@dotnetrepoman dotnetrepoman bot added this to the January 2025 milestone Jan 12, 2025
@dotnet-policy-service dotnet-policy-service bot added dotnet-fundamentals/svc community-contribution Indicates PR is created by someone from the .NET community. labels Jan 12, 2025
@sigmade
Copy link
Contributor Author

sigmade commented Jan 12, 2025

@dotnet-policy-service agree

@sigmade
Copy link
Contributor Author

sigmade commented Jan 14, 2025

@IEvangelist Could you please tell me what I'm doing wrong? An error appears Please fix OPS build warnings before merging

Copy link
Member

@IEvangelist IEvangelist left a comment

Choose a reason for hiding this comment

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

Let's get some of this feedback added and look at it again. Also, this is going to need to have a TOC.yml entry.

sigmade and others added 15 commits January 18, 2025 12:06
@sigmade
Copy link
Contributor Author

sigmade commented Jan 18, 2025

@IEvangelist Thanks for the edits! I accepted commit changes and also corrected TOC.yml

@IEvangelist
Copy link
Member

Hey there, @sigmade - sorry, still playing catch up...could you please address the merge conflict here and I'll have another review? This is starting to shape up nicely.

@sigmade
Copy link
Contributor Author

sigmade commented Mar 11, 2025

Hey there, @sigmade - sorry, still playing catch up...could you please address the merge conflict here and I'll have another review? This is starting to shape up nicely.

Hi! done

@IEvangelist
Copy link
Member

@sigmade - I'm confused, whenever I was providing suggestions—they were being resolved without any changes. Some of these things are required in order for this content. I'm making these suggestions to resolve issues in the content and address linting warnings.

@sigmade
Copy link
Contributor Author

sigmade commented Mar 20, 2025

@sigmade - I'm confused, whenever I was providing suggestions—they were being resolved without any changes. Some of these things are required in order for this content. I'm making these suggestions to resolve issues in the content and address linting warnings.

I just resolved it

@IEvangelist
Copy link
Member

@sigmade - I'm confused, whenever I was providing suggestions—they were being resolved without any changes. Some of these things are required in order for this content. I'm making these suggestions to resolve issues in the content and address linting warnings.

I just resolved it

Right, but selecting "Resolve" doesn't take any action on the suggestion. It just dismisses it.

@sigmade
Copy link
Contributor Author

sigmade commented Mar 20, 2025

@sigmade - I'm confused, whenever I was providing suggestions—they were being resolved without any changes. Some of these things are required in order for this content. I'm making these suggestions to resolve issues in the content and address linting warnings.

I just resolved it

Right, but selecting "Resolve" doesn't take any action on the suggestion. It just dismisses it.

It's my fault. I should have chosen - Commit suggestion

@IEvangelist IEvangelist merged commit 59c7691 into dotnet:main Mar 20, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community-contribution Indicates PR is created by someone from the .NET community. dotnet-fundamentals/svc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants