Avoid "OSX" in warning messages?

dotnet / platform-compat

Roslyn analyzer that finds usages of APIs that will throw PlatformNotSupportedException on certain platforms.

MIT License

278 stars 43 forks source link

Closed justinvp closed 6 years ago

justinvp commented 6 years ago

The use of "MacOSX" in the warnings looks dated.

Apple no longer uses "OS X". They call it "macOS" (yes, lowercase 'm', to align with the naming of iOS, watchOS, and tvOS).

Should these warnings use "macOS" or simply "Mac", instead of "MacOSX"?

terrajobst commented 6 years ago

Right now, I'm just ToString'ing the internal enum we use. Easy fix though. Would also address #62.