Skip to content

Updated README.md for Windows#4

Merged
pisapi1 merged 1 commit into
mainfrom
doc
Sep 5, 2025
Merged

Updated README.md for Windows#4
pisapi1 merged 1 commit into
mainfrom
doc

Conversation

@pisapi1

@pisapi1 pisapi1 commented Sep 4, 2025

Copy link
Copy Markdown
Collaborator

Added instructions for Step 1 for WindowsOS

Copilot AI review requested due to automatic review settings September 4, 2025 22:17

Copilot AI 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.

Pull Request Overview

This PR adds Windows-specific installation instructions for Poetry to the README.md file. The changes provide an alternative installation method using Scoop package manager for Windows users.

  • Adds a new section with step-by-step Windows installation instructions
  • Introduces Scoop package manager as the installation method for Windows
  • Provides PowerShell commands for setting up the development environment

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread README.md
Comment on lines +25 to +28
# To set execution policy
Set-ExecutionPolicy RemoteSigned -scope CurrentUser
# To download Scoop and by running the installer script for Scoop w/ elevated privileges
Invoke-Expression “& {$(Invoke-RestMethod get.scoop.sh)} -RunAsAdmin”

Copilot AI Sep 4, 2025

Copy link

Choose a reason for hiding this comment

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

The code block is marked as 'bash' but contains PowerShell commands. Change the language identifier to 'powershell' for proper syntax highlighting.

Copilot uses AI. Check for mistakes.
Comment thread README.md
**NOTE: Make sure to close out terminal to reflect changes**

3. Run below command to confirm Scoop installment by listing available commands on Scoop
```bash

Copilot AI Sep 4, 2025

Copy link

Choose a reason for hiding this comment

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

The code block is marked as 'bash' but contains a command that would be run in PowerShell on Windows. Change the language identifier to 'powershell' for consistency.

Copilot uses AI. Check for mistakes.
Comment thread README.md
Comment on lines +40 to +41
scoop install pipx
pipx ensurepath

Copilot AI Sep 4, 2025

Copy link

Choose a reason for hiding this comment

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

The code block is marked as 'bash' but contains commands that would be run in PowerShell on Windows. Change the language identifier to 'powershell' for consistency.

Copilot uses AI. Check for mistakes.
Comment thread README.md
```bash
# To set execution policy
Set-ExecutionPolicy RemoteSigned -scope CurrentUser
# To download Scoop and by running the installer script for Scoop w/ elevated privileges

Copilot AI Sep 4, 2025

Copy link

Choose a reason for hiding this comment

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

[nitpick] The comment has awkward phrasing. Consider revising to: '# Download and install Scoop with elevated privileges'.

Suggested change
# To download Scoop and by running the installer script for Scoop w/ elevated privileges
# Download and install Scoop with elevated privileges

Copilot uses AI. Check for mistakes.
Comment thread README.md

**NOTE: Make sure to close out terminal to reflect changes**

3. Run below command to confirm Scoop installment by listing available commands on Scoop

Copilot AI Sep 4, 2025

Copy link

Choose a reason for hiding this comment

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

The word 'installment' should be 'installation' in this context.

Suggested change
3. Run below command to confirm Scoop installment by listing available commands on Scoop
3. Run below command to confirm Scoop installation by listing available commands on Scoop

Copilot uses AI. Check for mistakes.
@pisapi1 pisapi1 merged commit 94af078 into main Sep 5, 2025
3 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.

3 participants