[MTA-6933] Visual Studio Code MTA plugin installation options#365
Conversation
📝 WalkthroughWalkthroughThe VS Code extension installation doc was rewritten to present two options—install the {ProductFullName} extension pack or use {ProductShortName} Core plus language-specific plugins—adds a Developer Preview warning for language extensions, and moves external links into an Additional resources subsection. ChangesVS Code Extension Installation Documentation
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Nitpick comments (1)
docs/topics/vscode/proc_installing-vscode-extension.adoc (1)
15-15: ⚡ Quick winReconsider backticks around "Extension pack".
Backticks typically denote code or monospace text in AsciiDoc. Since "Extension pack" is a product name rather than code, consider removing the backticks or using appropriate emphasis markup if needed.
📝 Suggested adjustment
-:FeatureName: `Extension pack`, `C#`, `Javascript`, and `Go` extensions +:FeatureName: Extension pack, `C#`, `JavaScript`, and `Go` extensions🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@docs/topics/vscode/proc_installing-vscode-extension.adoc` at line 15, The frontmatter attribute :FeatureName: currently wraps "Extension pack" in backticks, which are code-style in AsciiDoc; update the attribute value to remove the backticks around Extension pack (e.g., change `Extension pack` to Extension pack) or replace them with appropriate emphasis (e.g., _Extension pack_ or *Extension pack*) so the product name is not rendered as code; edit the :FeatureName: attribute where it's defined to apply this change.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/topics/vscode/proc_installing-vscode-extension.adoc`:
- Line 11: The phrase uses incorrect capitalization for proper nouns: replace
occurrences of "Javascript" and "javascript" with "JavaScript" and change
"node.js" to "Node.js" in the line containing "* `Javascript` to analyze
`javascript` and `node.js` applications" so the docs read "* `JavaScript` to
analyze `JavaScript` and `Node.js` applications".
- Line 6: The sentence uses the awkward phrase "optimize the extension memory";
update the wording in the sentence that starts "Install the {ProductFullName}
extension pack..." to use clearer language such as "optimize memory usage" or
"reduce memory footprint" (e.g., "Use the extension pack to analyze in all
supported languages or the *Core* and a language-specific plugin to reduce
memory footprint.") so the intent is clear and reads naturally.
---
Nitpick comments:
In `@docs/topics/vscode/proc_installing-vscode-extension.adoc`:
- Line 15: The frontmatter attribute :FeatureName: currently wraps "Extension
pack" in backticks, which are code-style in AsciiDoc; update the attribute value
to remove the backticks around Extension pack (e.g., change `Extension pack` to
Extension pack) or replace them with appropriate emphasis (e.g., _Extension
pack_ or *Extension pack*) so the product name is not rendered as code; edit the
:FeatureName: attribute where it's defined to apply this change.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: b65e775c-cf87-4209-8f64-cf41a6b962c9
📒 Files selected for processing (1)
docs/topics/vscode/proc_installing-vscode-extension.adoc
|
@mpershina Can you please give this installation topic a quick review? Thanks! |
mpershina
left a comment
There was a problem hiding this comment.
Minor changes needed.
Done, thanks! |
Signed-off-by: Prabha Kylasamiyer Sundara Rajan <pkylasam@pkylasam-thinkpadp16vgen1.bengluru.csb>
6e7411b to
fe4b041
Compare
|
@Pkylas007 - please do a sync and publish. thanks |
|
Version
JIRA
Preview
Summary by CodeRabbit