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

MSBuild common properties #3843

Merged
merged 23 commits into from
Jul 4, 2024
Merged

Conversation

s2quake
Copy link
Contributor

@s2quake s2quake commented Jun 20, 2024

This PR corresponds to step2 of issue #3830 and includes.

  • Common properties for Directory.Build.props files in src, tools, and test are defined for each path.
  • Directory.Build.props in the solutions path defines common properties for the entire project.
  • In a typical environment, the TargetFramework property is net6.0.
  • The TargetFrameworks used when running the dotnet pack command is also net6.0, but the TargetFrameworks of the project in src has the following values: netstandard2.0;netstandard2.1;netcoreapp3.1;net6.0.
  • Because the default TargetFramework value is net6.0, some of the code has been branched using #if #endif.
  • Fixed an issue with paths when running CI scripts.

@s2quake s2quake force-pushed the chore/common-properties branch 2 times, most recently from e809dda to cd21fb3 Compare June 21, 2024 05:14
@s2quake s2quake self-assigned this Jun 21, 2024
@s2quake s2quake added the no changelog Bypasses changelog check label Jun 21, 2024
@s2quake s2quake force-pushed the chore/common-properties branch 3 times, most recently from 8d9e289 to 070e243 Compare June 21, 2024 09:07
@s2quake s2quake mentioned this pull request Jun 24, 2024
8 tasks
@s2quake s2quake force-pushed the chore/common-properties branch 5 times, most recently from ad443f0 to 5d09b93 Compare June 25, 2024 09:17
@s2quake s2quake closed this Jun 26, 2024
@s2quake s2quake reopened this Jun 26, 2024
@s2quake s2quake force-pushed the chore/common-properties branch 2 times, most recently from 9173b63 to ea25c84 Compare June 26, 2024 04:09
@s2quake s2quake marked this pull request as ready for review June 26, 2024 04:52
@OnedgeLee OnedgeLee mentioned this pull request Jun 27, 2024
riemannulus
riemannulus previously approved these changes Jul 1, 2024
@s2quake s2quake requested a review from riemannulus July 3, 2024 00:40
@s2quake s2quake removed the no changelog Bypasses changelog check label Jul 3, 2024
@s2quake s2quake merged commit 1bed9c0 into planetarium:main Jul 4, 2024
22 checks passed
@s2quake s2quake deleted the chore/common-properties branch July 4, 2024 06:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants