-
Notifications
You must be signed in to change notification settings - Fork 353
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
APICompat is showing noisy diffs where it shouldn't (regarding Parameter Modifiers) #2136
Comments
Simple repro:
Suspect a bad comparator here: arcade/src/Microsoft.DotNet.ApiCompat/Rules/Compat/ParameterModifiersCannotChange.cs Lines 66 to 72 in 767154a
Perhaps that's not doing an actual equality check when evaluating the union. |
This appears to have been fixed by @dougbu in https://github.com/dotnet/arcade/pull/4585/files#diff-ab264297f1e0d7296c8d69384a224e589d95ced59d203663d968dcc20f7d1e13R68 Note that the error may still appear if ApiCompat is not able to see that the types on both sides are equivalent, so in my repro above it will still fail unless I pass
|
These two netfx dlls are showing up in ApiCompat runs even when they are identical:
The diffs themselves are all around
ParameterModifiersCannotChange
and seem non-actionable anyway:Comparing all netfx refs
corefx\bin\ref\netfx
,net45+
, ...See: dotnet/corefx#35557
To reproduce:
dotnet "Microsoft.DotNet.ApiCompat.dll" "bin\ref\netfx" -i "bin\ref\netfx"
cc @ericstj
The text was updated successfully, but these errors were encountered: