Skip to content

docs(vscode): update stale ms.date in .vscode/README.md - #3205

Merged
Jamie Kim (jkim323) merged 2 commits into
microsoft:mainfrom
Fire162:docs/update-vscode-readme-msdate
Oct 16, 2026
Merged

Jamie Kim (jkim323) merged 2 commits into
microsoft:mainfrom
Fire162:docs/update-vscode-readme-msdate

Conversation

@Fire162

@Fire162 Abhinav Maurya (Fire162) commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

Pull Request

Description

Updates the ms.date frontmatter in .vscode/README.md from 2026-07-08
to 2026-10-12.

No MCP server configuration guidance, authentication options,
configuration examples, or references are changed.

Related Issue(s)

Fixes #3188

Type of Change

Code & Documentation:

  • Bug fix (non-breaking change fixing an issue)
  • New feature (non-breaking change adding functionality)
  • Breaking change (fix or feature causing existing functionality to change)
  • Documentation update

Infrastructure & Configuration:

  • GitHub Actions workflow
  • Linting configuration (markdown, PowerShell, etc.)
  • Security configuration
  • DevContainer configuration
  • Dependency update

AI Artifacts:

  • Reviewed contribution with hve-builder and addressed all actionable findings
  • Copilot instructions (.github/instructions/*.instructions.md)
  • Copilot prompt (.github/prompts/*.prompt.md)
  • Copilot agent (.github/agents/*.agent.md)
  • Copilot skill (.github/skills/*/SKILL.md)
  • Copilot hook (.github/hooks/*/*.json)
  • Eval spec added/updated for changed AI artifacts (evals/)

Other:

  • Script/automation (.ps1, .sh, .py)
  • Other (please describe):

Testing

Reviewed the diff and confirmed that only the ms.date value changes.
Local validation commands have not been verified for this description.

Checklist

Required Checks

  • Documentation is updated (if applicable)
  • Files follow existing naming conventions
  • Changes are backwards compatible (if applicable)
  • Tests added for new functionality (if applicable)

No new functionality is introduced; additional tests are not applicable.

AI Artifact Contributions

  • Used hve-builder review mode to review contribution
  • Addressed all actionable findings from the hve-builder review
  • Verified contribution follows common standards and type-specific requirements

Not applicable: no AI artifacts are changed.

Required Local Checks

The following local-safe validation commands must pass before merging:

  • Local validation aggregate: npm run validate:local
  • Documentation validation (if docs changed): npm run validate:docs
  • Spell checking: npm run spell-check
  • Link validation: npm run lint:md-links

Security Considerations

  • This PR does not contain any sensitive or NDA information
  • Any new dependencies have been reviewed for security issues
  • Security-related scripts follow the principle of least privilege

No dependencies or security-related scripts are added or changed.

Additional Notes

This is a documentation metadata-only update; no VS Code or MCP
configuration files are modified. The Sample Prompts section is omitted
because no AI artifacts are changed.

@Fire162
Abhinav Maurya (Fire162) requested a review from a team as a code owner October 12, 2026 19:03

@jkim323 Jamie Kim (jkim323) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for the contribution, Abhinav Maurya (@Fire162) and welcome!

Could you please update the PR description to follow the repository's PR template?

Thank you!

@codecov-commenter

Codecov Comments Bot (codecov-commenter) commented Oct 12, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.98%. Comparing base (d540a54) to head (f780abd).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #3205      +/-   ##
==========================================
+ Coverage   81.90%   84.98%   +3.07%     
==========================================
  Files         125      193      +68     
  Lines       17625    39428   +21803     
  Branches       63      201     +138     
==========================================
+ Hits        14436    33508   +19072     
- Misses       3178     5849    +2671     
- Partials       11       71      +60     
Flag Coverage Δ
docusaurus ?
pester 86.32% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 107 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The Configuration note still tells readers to run npx @modelcontextprotocol/server-github, but npm view @modelcontextprotocol/server-github deprecated reports the package as no longer supported (last version 2025.4.8). The committed mcp.json.sample already uses the hosted server at https://api.githubcopilot.com/mcp/, and it also ships workiq and figma entries that the Overview never mentions. Those lines need updating before the page is marked as reviewed.

@jkim323 Jamie Kim (jkim323) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Updated PR template to prevent further drift

@jkim323

Copy link
Copy Markdown
Contributor

The Configuration note still tells readers to run npx @modelcontextprotocol/server-github, but npm view @modelcontextprotocol/server-github deprecated reports the package as no longer supported (last version 2025.4.8). The committed mcp.json.sample already uses the hosted server at https://api.githubcopilot.com/mcp/, and it also ships workiq and figma entries that the Overview never mentions. Those lines need updating before the page is marked as reviewed.

Thanks for flagging these discrepancies, Mohammed Alkindi (@MohammedAlkindi). This PR is scoped to updating stale ms.date metadata; the deprecated package reference and missing Overview entries should be addressed in a follow-up. I've updated the description to clarify that this is a date-only change, rather than claim the technical content has been verified.

@jkim323
Jamie Kim (jkim323) added this pull request to the merge queue Oct 16, 2026
Merged via the queue into microsoft:main with commit 4b6b7ed Oct 16, 2026
147 checks 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.

docs: Update stale documentation - .vscode/README.md

5 participants