-
Notifications
You must be signed in to change notification settings - Fork 509
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
Violation of some rules only generate IntelliSense errors, but not build errors #1718
Comments
Do you see any AD0001 instances in your diagnostic build output? |
No. I couldn't find anything weird in the build output. Here it is: http://pastebin.com/Rz5g9rgs |
I notice the same behavior for the following rules: |
@mzervos Thanks, that's very helpful. We recently fixed similar issues in a number of spacing rules. Currently I'm not sure what's causing the discrepancy for the rules you identified but we'll figure it out! 😄 |
@mzervos Do you have SA1652 disabled? If so, can you make sure to enable that and see if it reports warnings during the build? |
@sharwell Yes, SA1652 is disabled. If I enabled it, the other rules cause errors on build as well. |
This is not possible. SA1652 should not be disabled, even if you are not using the output files. See the documentation for SA1652 for more information. |
How are they enabled and detected on IntelliSense though without having SA1652 turned on? |
I have no idea actually. It could be that Roslyn uses special settings for the IntelliSense parser in order to properly apply syntax highlighting and other editor features inside of documentation comments, but these are disabled when compiling unless the XML output is enabled. I'm going to mark this as external since it's other case where the underlying cause of dotnet/roslyn#2773 is causing problems. |
Some rules (like SA1642) produce InitelliSense errors but the project builds successfully without reporting those errors. I'm using beta15 version.
I don't see any warnings/errors on msbuild logs (i have set verbosity to Diagnostic)
The text was updated successfully, but these errors were encountered: