-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
Add testing platform architecture #41735
Add testing platform architecture #41735
Conversation
I don't see the image and the mermaid rendering correctly is it related only to the "preview"? |
@IEvangelist Are our docs supporting mermaid? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm still not done with this review, it's taking a really long time - been working on it for a while now. I'm going to comment on it, and return when I can focus a bit more. But for now, here's what I have.
docs/core/testing/unit-testing-platform-architecture-capabilities.md
Outdated
Show resolved
Hide resolved
docs/core/testing/unit-testing-platform-architecture-capabilities.md
Outdated
Show resolved
Hide resolved
docs/core/testing/unit-testing-platform-architecture-capabilities.md
Outdated
Show resolved
Hide resolved
docs/core/testing/unit-testing-platform-architecture-capabilities.md
Outdated
Show resolved
Hide resolved
docs/core/testing/unit-testing-platform-architecture-extensions.md
Outdated
Show resolved
Hide resolved
docs/core/testing/unit-testing-platform-architecture-extensions.md
Outdated
Show resolved
Hide resolved
docs/core/testing/unit-testing-platform-architecture-extensions.md
Outdated
Show resolved
Hide resolved
docs/core/testing/unit-testing-platform-architecture-extensions.md
Outdated
Show resolved
Hide resolved
docs/core/testing/unit-testing-platform-architecture-extensions.md
Outdated
Show resolved
Hide resolved
Co-authored-by: David Pine <david.pine@microsoft.com>
Co-authored-by: David Pine <david.pine@microsoft.com>
Thank you so much for everything you have reviewed already! We know this is a super big change, I have read it a few times and I am still not 100% happy with the results of what we wrote nor the structure but it's not always easy to get the abstraction of knowing all the implementation details. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are still a few things that we'll need to update, this is starting to shape up though. Let's consider this 2/3 in terms of reviews. 🤓
docs/core/testing/unit-testing-platform-architecture-extensions.md
Outdated
Show resolved
Hide resolved
docs/core/testing/unit-testing-platform-architecture-extensions.md
Outdated
Show resolved
Hide resolved
docs/core/testing/unit-testing-platform-architecture-extensions.md
Outdated
Show resolved
Hide resolved
docs/core/testing/unit-testing-platform-architecture-extensions.md
Outdated
Show resolved
Hide resolved
Co-authored-by: David Pine <david.pine@microsoft.com>
Co-authored-by: David Pine <david.pine@microsoft.com>
Co-authored-by: David Pine <david.pine@microsoft.com>
I've checked back on this one a few times, there's still a lot of suggestions that haven't been resolved. I'll come back once those are addressed. /cc @Evangelink |
@IEvangelist Yes sorry! I have been pretty busy with other priorities, I'll get back here early next week! |
No worries, I completely understand. I just wanted to make sure I wasn't missing anything. I'm off all next week, so I'll check back the week of 8/12. Thank you, my friend! @Evangelink |
Co-authored-by: David Pine <david.pine@microsoft.com>
Co-authored-by: David Pine <david.pine@microsoft.com>
…s.md Co-authored-by: David Pine <david.pine@microsoft.com>
Co-authored-by: David Pine <david.pine@microsoft.com>
Hey @IEvangelist, could we resume work on this PR? We have finished integration with xUnit v3, NUnit, and we are getting close to finishing with F# Expecto. |
@IEvangelist Could we resume work on this PR when you have some time? I think I have handled all the comments you made to me. |
We need to address the build warnings, see here. Then, let's ensure that all the images are cropped nicely, then we should be able to merge. |
@IEvangelist PR is green :) A HUGE thank you for your support through this massive update. |
Summary
Move and reorganize the architecture/extensibility documentation for the new testing platform.
cc @MarcoRossignoli
Internal previews