Skip to content

Conversation

@stakx
Copy link
Member

@stakx stakx commented Dec 28, 2022

It appears that the .NET compilers previously left this custom attribute property empty, but have now started setting it.

It appears that the .NET compilers previously left this custom attribute
property empty, but have now started setting it.
Our `PublicApi` verification test passes on AppVeyor while it fails on
GitHub Actions. One difference between the two is that AppVeyor uses v7
of the .NET SDK, while GitHub Actions uses v6.

Installing both .NET 6 and 7 will hopefully resolve the `PublicApi` test
failure with GitHub Actions builds.
@stakx
Copy link
Member Author

stakx commented Dec 28, 2022

What is this new devilry?

Took me a moment to figure out what went wrong here: apparently, .NET tooling started to provide a value for FrameworkDisplayName, except the .NET 6 SDK, which left it empty. This has apparently been fixed in dotnet/sdk#24656, and if we use the .NET 7 SDK when targeting .NET 6, the problem goes away.

@stakx stakx force-pushed the update-ref-contract-files branch from a660274 to 3f7a499 Compare December 28, 2022 11:55
@stakx stakx merged commit 9029d9f into castleproject:master Dec 28, 2022
@stakx stakx deleted the update-ref-contract-files branch December 28, 2022 12:08
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.

1 participant