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

DOCS-7907 Visual Studio Documentation Update #23083

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

alai97
Copy link
Contributor

@alai97 alai97 commented May 9, 2024

What does this PR do? What is the motivation?

Adds IDE Plugins section to the Code Analysis TOC + a Static Analysis section in the Visual Studio doc page.

Chatted with Mark about it on Slack. We are waiting for a screenshot of the working example (in dark mode) to be replaced.

Merge instructions

  • [] Please merge after reviewing

Additional notes

Adds IDE Plugins section to the Code Analysis TOC + a Static Analysis section in the Visual Studio doc page.
@alai97 alai97 requested a review from a team as a code owner May 9, 2024 03:03
@github-actions github-actions bot added the Architecture Everything related to the Doc backend label May 9, 2024

Static Analysis supports scanning across multiple programming languages. For a complete list, see [Static Analysis Rules][15]. For file types belonging to supported languages, issues are shown in the source code editor using Visual Studio's code inspection system, and suggested fixes can be applied directly:

{{< img src="/developers/ide_plugins/visual_studio/TBD.png" alt="A static analysis rule violation and recommended fix." style="width:100%;" >}}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

To be replaced: image in dark mode @markazerdd

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@AlonAm to add the screenshot in here

Copy link
Contributor

Choose a reason for hiding this comment

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

Added a new screenshot, let me know what you think, thanks

Copy link
Contributor

github-actions bot commented May 9, 2024

Preview links (active after the build_preview check completes)

Modified Files

@alai97 alai97 requested a review from markazerdd May 9, 2024 04:27
Copy link
Contributor

@hestonhoffman hestonhoffman left a comment

Choose a reason for hiding this comment

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

LGTM. Left a suggestion for content reuse

Comment on lines +3745 to +3749
- name: IDE Plugins
url: developers/ide_plugins
parent: code_analysis
identifier: code_analysis_ide_plugins
weight: 5
Copy link
Contributor

Choose a reason for hiding this comment

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

Have you tried a full-page reuse here? Might be a good use case for it.

@github-actions github-actions bot added the Images Images are added/removed with this PR label May 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Architecture Everything related to the Doc backend Images Images are added/removed with this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants