-
Notifications
You must be signed in to change notification settings - Fork 42
Closed
Description
Sad news. Does this mean that other projects also have this problem? I see readme in graphql-dotner but not parser 🤔 . Looks a bit weird. How about just move property out of Condition?
<PropertyGroup>
<PackageReadmeFile>README.md</PackageReadmeFile>
</PropertyGroup>The same for AnalysisMode. I understand it requires a bit more work but it should not be difficult. I propose to add CA1707 into NoWarn in Tests.props . After that I see only ~60 issues to fix 😉 . I think this is better than jumping over different order of conditions in MSBuild.
Originally posted by @sungam3r in graphql-dotnet/server#897 (comment)
Metadata
Metadata
Assignees
Labels
No labels