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
[4.0] Fix numerous binder errors #781
Conversation
|
||
sw.WriteLine("// ReSharper disable InconsistentNaming"); | ||
sw.WriteLine("#pragma warning disable SA1139 // Use literal suffix notation instead of casting"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this required? I tried looking for places which break this rule but couldn't find any.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's mainly for the enumerations and their members - it shows up as a bunch of warnings otherwise.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right, that's fine then - I did know what it was for, but couldn't find any examples where it would show warnings. Figured it couldn't hurt to ask
This PR fixes a number of issues raised in the binder code, allowing it to build and run again. On top of that, it also improves the API generation step, ensuring that generated code doesn't raise any StyleCop errors or warnings.
This greatly reduces the number of errors in the solution, and lays the groundwork for future improvements to the binder.
There are still some documentation-related issues remaining.