-
Notifications
You must be signed in to change notification settings - Fork 648
Update xunit.v3 to 3.0.0-pre.25 #9894
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
Conversation
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.
Pull Request Overview
This PR updates the xUnit dependency version in Versions.props from 2.0.3 to 3.0.0-pre.25 to address "go to test" issues and improve performance.
- Upgrades xUnit from a stable to a prerelease version.
- Aims to resolve test discovery performance and behavior issues.
Comments suppressed due to low confidence (1)
eng/Versions.props:15
- Ensure that all test projects and build configurations are validated against this prerelease version to avoid potential discrepancies with xUnit v3 APIs.
<XunitV3Version>3.0.0-pre.25</XunitV3Version>
#9895 needs to be merged first. |
|
@davidfowl @captainsafia @joperezr @radical I think we should consider taking this change in the interests for productivity improvements. Combined with the updated MTP library that just merged into main, this prerelease package fixes the Go to Test experience in VSCode which I know that @davidfowl, @captainsafia and I have been struggling with. I think its safe to take the dependency because I don't think it'll impact any of the packages/templates that we ship. |
Co-authored-by: Dan Moseley <danmose@microsoft.com>
Description
See #9876 (comment).
@mitchdenny Can you verify if this resolves the "go to test" issues? It's then up to you to wait for a stable version, or merging the prerelease. Also note that this should bring a bit of performance benefit for discovery.
Fixes # (issue)
Checklist
<remarks />
and<code />
elements on your triple slash comments?doc-idea
templatebreaking-change
templatediagnostic
template