Skip to content

Conversation

@ThomasGoulet73
Copy link
Contributor

@ThomasGoulet73 ThomasGoulet73 commented Jun 6, 2025

Description

This reverts commit 2ff7cad (#10338). The linked issue (dotnet/runtime#111825) was closed when the fix (dotnet/runtime#114651) was merged in the runtime. The fix flowed in this repo and I'm no longer able to reproduce it locally.

Customer Impact

None, tests only.

Regression

No.

Testing

Ran the tests locally and tested the initial test I used when I pushed the workaround.

Risk

Low to none, tests only.

Microsoft Reviewers: Open in CodeFlow

@ThomasGoulet73 ThomasGoulet73 requested a review from a team as a code owner June 6, 2025 23:20
@dotnet-policy-service dotnet-policy-service bot added PR metadata: Label to tag PRs, to facilitate with triage Community Contribution A label for all community Contributions labels Jun 6, 2025
@codecov
Copy link

codecov bot commented Jun 6, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 2.98754%. Comparing base (416cfa6) to head (adec62f).
Report is 2 commits behind head on main.

Additional details and impacted files
@@                Coverage Diff                @@
##               main     #10919         +/-   ##
=================================================
- Coverage   2.98812%   2.98754%   -0.00058%     
=================================================
  Files          3320       3319          -1     
  Lines        664900     664894          -6     
  Branches      74675      74674          -1     
=================================================
- Hits          19868      19864          -4     
+ Misses       644851     644849          -2     
  Partials        181        181                 
Flag Coverage Δ
Debug 2.98754% <ø> (-0.00058%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@harshit7962 harshit7962 merged commit 9beecf3 into dotnet:main Jun 9, 2025
8 checks passed
@harshit7962
Copy link
Member

Thank you for the follow-up @ThomasGoulet73

@github-actions github-actions bot locked and limited conversation to collaborators Jul 9, 2025
@ThomasGoulet73 ThomasGoulet73 deleted the revert-unit-tests-native-dependencies-fix branch August 16, 2025 04:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Community Contribution A label for all community Contributions PR metadata: Label to tag PRs, to facilitate with triage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants