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

Rework the wiki #6383

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Rework the wiki #6383

wants to merge 3 commits into from

Conversation

Youssef1313
Copy link
Member

@Youssef1313 Youssef1313 commented Dec 22, 2022

For easier review, the first commit is an exact port of https://github.com/dotnet/roslyn-analyzers/wiki.

Later commits are the changes done to the Wiki.

If this PR gets merged, https://github.com/dotnet/roslyn-analyzers/wiki will need to be updated to only have a single link to the wiki in the repository.

You can also choose to close the PR and move these changes directly to the Wiki.

The benefit of having this as part of the repository is to enable community contributions.

Fixes #5234

docs/wiki.md Outdated

## Install Instructions
You can download the FXCop live code analyzers as a [NuGet package](https://www.nuget.org/packages/Microsoft.CodeAnalysis.FxCopAnalyzers/) or as a Visual Studio extension ([VS2017](https://marketplace.visualstudio.com/items?itemName=VisualStudioPlatformTeam.MicrosoftCodeAnalysis2017) or [VS2019](https://marketplace.visualstudio.com/items?itemName=VisualStudioPlatformTeam.MicrosoftCodeAnalysis2019)).
We've rebuilt the most popular FxCop rules and additional more rules as live analyzers to help you detect problems in your code and make it super easy for you to fix them on the spot with quick fixes. You can get the analyzers via [`Microsoft.CodeAnalysis.NetAnalyzers` NuGet package](https://www.nuget.org/packages/Microsoft.CodeAnalysis.NetAnalyzers) or from the .NET SDK. This gives you the same great code analysis as before with FXCop but now the analysis is live as-you-type and has quick-fixes.
Copy link
Member

Choose a reason for hiding this comment

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

I think we should avoid mentioning specific NuGet package, or .NET SDK or any acquisition modes here and instead just add a help link to https://learn.microsoft.com/dotnet/fundamentals/code-analysis/overview?tabs=net-7 which has all the relevant information.

docs/wiki.md Outdated
## Apply a Code Fix for a Rule
Place your cursor in the span of the squiggle/dotted line and press `Ctrl+.` to trigger the lightbulb menu. From the lightbulb, you will see all available quick actions.
> Note: not all 100 code analysis rules in this extension have fixes associated with them.
## Install instructions
Copy link
Member

Choose a reason for hiding this comment

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

Again, I think we should avoid repeating Install related information in our wiki as it is already covered in .NET code analysis docs.

docs/wiki.md Outdated

## Apply a code fix for a rule

Place your cursor in the span of the squiggle/dotted line and press `Ctrl+.` to trigger the lightbulb menu. From the lightbulb, you will see all available quick actions.
Copy link
Member

@mavasani mavasani Jan 3, 2023

Choose a reason for hiding this comment

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

Please avoid all the text here and instead just add a help link to https://learn.microsoft.com/visualstudio/ide/quick-actions

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update Wiki for several issues
2 participants