Skip to content

Conversation

@benrr101
Copy link
Contributor

Description: I spent far too long trying to track down completely unrelated errors that were resolved by this. The Directory.Build.props file two directories up includes definitions for TargetFrameworks, as well as in the ExtUtilities project. So when attempting certain builds from the IDE, it will throw errors that net9.0 is specified >1 and refuse to build. Removing the line in the csproj file resolves the issue.

Testing: Local builds are working again, will wait to see if it causes CI issues.

@benrr101 benrr101 added 🐛 Bug! Area\Tests Issues that are targeted to tests or test projects Area\Engineering Use this for issues that are targeted for changes in the 'eng' folder or build systems. labels Feb 24, 2025
@benrr101 benrr101 requested review from a team and mdaigle February 24, 2025 20:34
@codecov
Copy link

codecov bot commented Feb 24, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 72.82%. Comparing base (0be2756) to head (4b71592).
Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3178   +/-   ##
=======================================
  Coverage   72.81%   72.82%           
=======================================
  Files         282      282           
  Lines       59112    59112           
=======================================
+ Hits        43045    43050    +5     
+ Misses      16067    16062    -5     
Flag Coverage Δ
addons 92.58% <ø> (ø)
netcore 75.53% <ø> (+<0.01%) ⬆️
netfx 71.15% <ø> (+<0.01%) ⬆️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@benrr101 benrr101 merged commit cefeabc into main Feb 25, 2025
130 checks passed
@benrr101 benrr101 deleted the dev/russellben/framework-dup branch February 25, 2025 17:17
@mdaigle mdaigle added this to the 7.0-preview1 milestone Mar 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area\Engineering Use this for issues that are targeted for changes in the 'eng' folder or build systems. Area\Tests Issues that are targeted to tests or test projects

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants