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

Update InspectCode to 2022.3.3 and enable C# 11 #23570

Merged
merged 4 commits into from
May 17, 2023

Conversation

huoyaoyuan
Copy link
Contributor

@huoyaoyuan huoyaoyuan commented May 16, 2023

Prerequisite of #23282

Both 2023.1 and 2023.2EAP crashes hardly. I've test that 2022.3 version can inspect #23282 successfully.

The suppression is for reporting $(MSBuildThisFileDirectory) as unknown property. I don't think this feature is correct in any form because $(MSBuildThisFileDirectory) is well-known for a long term.

@bdach
Copy link
Collaborator

bdach commented May 16, 2023

The suppression is for reporting $(MSBuildThisFileDirectory) as unknown property.

The relevant youtrack report for this seems to be https://youtrack.jetbrains.com/issue/RSRP-490805 https://youtrack.jetbrains.com/issue/RSRP-490368. I've added an inline comment next to the suppression for reference.

bdach
bdach previously approved these changes May 16, 2023
@peppy peppy merged commit ab2bd12 into ppy:master May 17, 2023
@huoyaoyuan huoyaoyuan deleted the inspect-code-cs11 branch May 17, 2023 03:04
peppy added a commit that referenced this pull request May 17, 2023
@peppy
Copy link
Member

peppy commented May 17, 2023

Sorry but had to revert this (dc421bd). Can no longer build the game with .NET 6 SDK, so we'll need to wait a bit longer before making this change.

osu on  master via .NET v6.0.408
❯ dotnet build
MSBuild version 17.3.2+561848881 for .NET
  Determining projects to restore...
  Restored /Users/dean/Projects/osu/osu.Game.Rulesets.Osu.Tests.Android/osu.Game.Rulesets.Osu.Tests.Android.csproj (in 7.32 sec).
  Restored /Users/dean/Projects/osu/osu.Game.Tests.Android/osu.Game.Tests.Android.csproj (in 7.32 sec).
  Restored /Users/dean/Projects/osu/osu.Game.Rulesets.Catch.Tests.Android/osu.Game.Rulesets.Catch.Tests.Android.csproj (in 7.32 sec).
  Restored /Users/dean/Projects/osu/osu.Game.Rulesets.Mania.Tests.Android/osu.Game.Rulesets.Mania.Tests.Android.csproj (in 7.32 sec).
  Restored /Users/dean/Projects/osu/osu.Game.Rulesets.Taiko.Tests.Android/osu.Game.Rulesets.Taiko.Tests.Android.csproj (in 7.32 sec).
  Restored /Users/dean/Projects/osu/osu.Android/osu.Android.csproj (in 7.41 sec).
  22 of 28 projects are up-to-date for restore.
CSC : error CS1617: Invalid option '11.0' for /langversion. Use '/langversion:?' to list supported values. [/Users/dean/Projects/osu/osu.Game/osu.Game.csproj]

Build FAILED.

CSC : error CS1617: Invalid option '11.0' for /langversion. Use '/langversion:?' to list supported values. [/Users/dean/Projects/osu/osu.Game/osu.Game.csproj]
    0 Warning(s)
    1 Error(s)

Time Elapsed 00:00:11.64

@huoyaoyuan
Copy link
Contributor Author

Can no longer build the game with .NET 6 SDK

Hm, doesn't CI specifies the lowest SDK version we want to support?

BTW there are some further optimizations that requires .NET 7. How should the update be driven?

@peppy
Copy link
Member

peppy commented May 17, 2023

Start at framework side and work outwards. Make sure to benchmark said optimisations to see how much they benefit us, as that may change the priority of switching to 7.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants