Skip to content

Conversation

@martinpeck
Copy link
Member

Both bash and powershell examples were identical, and neither had bash or powershell specific instructions.
This resulted in an apparent duplication when the docs were viewed.
Replace both of these with a single console example.

Pull Request

Description

Related Issue(s)

Fixes #95"

Type of Change

Select all that apply:

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 prompt-builder chatmode and addressed all feedback
  • Copilot instructions (.github/instructions/*.instructions.md)
  • Copilot prompt (.github/prompts/*.prompt.md)
  • Copilot chatmode (.github/chatmodes/*.chatmode.md)

Required Checks

  • Documentation is updated (if applicable)
  • Files follow existing naming conventions
  • Changes are backwards compatible (if applicable)

Required Automated Checks

The following validation commands must pass before merging:

  • Markdown linting: npm run lint:md
  • Spell checking: npm run spell-check
  • Frontmatter validation: npm run lint:frontmatter
  • Link validation: npm run lint:md-links
  • PowerShell analysis: npm run lint:ps

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

Additional Notes

Author has MS alias: mpeck

Both bash and powershell examples were identical, and neither had bash or powershell specific instructions. Replace both of these with a single `console` example.
Copilot AI review requested due to automatic review settings November 26, 2025 15:45
Copilot finished reviewing on behalf of martinpeck November 26, 2025 15:47
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR consolidates duplicate code blocks in the getting-started documentation by replacing separate bash and powershell examples with a single console code block. The change eliminates redundancy since the commands (cd and git clone) are identical across both shells and don't require shell-specific syntax.

Key Changes

  • Removed duplicate bash and powershell code blocks showing identical git clone commands
  • Replaced with single console code block for cross-platform compatibility

Copilot AI review requested due to automatic review settings November 26, 2025 21:32
Copilot finished reviewing on behalf of WilliamBerryiii November 26, 2025 21:34
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Copy link

@bindsi bindsi left a comment

Choose a reason for hiding this comment

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

thanks!

@martinpeck
Copy link
Member Author

The build fails related to linting appear to be from other merged commits.
Want me to fix them in my branch, or are they being addressed elsewhere?

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.

[Issue]: Getting Started Doc Has Duplicate Example

3 participants