Skip to content
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

Provide images for documentation about Roslyn Analyzers #4063

Merged
merged 2 commits into from
Jun 26, 2024

Conversation

kant2002
Copy link
Contributor

Also

  • Add fix for the code fix broken when static constructor is present in the class.
  • Add debugging setup for the Csla.Analyzers project

Closes #1519
Closes #4061

Also
- Add fix for the code fix broken when static constructor is present in the class.
- Add debugging setup for the Csla.Analyzers project

Closes MarimerLLC#1519
Closes MarimerLLC#4061

![Crash of the analyzer](images/AnalyzerCrash.png "Crash of the analyzer")

Unfortunately, Visual Studio doesn't make it easy to get crash information on an analyzer. If you want provide more information in addition to stack trace, or something is more complicated, you can launch `devenv.exe` with the `/log` switch, and see if the log file contains any meaningful information. Also, please log an issue on the CSLA site if you do find problems with an analyzer, and we'll do our best to resolve the problem.

## Analyzer Proposals

If you have an idea for an analyzer that would be beneficial for developers using CSLA, please propose it in the forums (https://github.com/MarimerLLC/cslaforum/issues). Tag it with "feature discussion". If it's decided that it's a good idea to implement, it'll be moved to the CSLA issues list.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@@ -86,10 +86,11 @@ public override async Task RegisterCodeFixesAsync(CodeFixContext context)
{
var classSymbol = model.GetDeclaredSymbol(classNode);

//System.Diagnostics.Debugger.Launch();
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove commented code

@rockfordlhotka rockfordlhotka merged commit 26b9556 into MarimerLLC:main Jun 26, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Code fix for public constructor seems to be broken Update analyzer doc file
2 participants