Skip to content

Conversation

@Endkind
Copy link
Member

@Endkind Endkind commented Oct 15, 2025

  • maintenance: bump pre-commit tool versions (pre-commit-hooks v6.0.0, black 25.9.0, isort 6.1.0)
  • maintenance: replace Windows setup.bat with setup.cmd and add setup.ps1
  • maintenance: harden setup.sh (set -euo pipefail), improve messages, install order, and run pre-commit install
  • maintenance: standardize setup flow (.venv creation, activate, upgrade pip, install pre-commit & hooks, install requirements if present)

…, black 25.9.0, isort 6.1.0)

- maintenance: replace Windows setup.bat with setup.cmd and add setup.ps1
- maintenance: harden setup.sh (set -euo pipefail), improve messages, install order, and run pre-commit install
- maintenance: standardize setup flow (.venv creation, activate, upgrade pip, install pre-commit & hooks, install requirements if present)
@Endkind Endkind requested a review from Copilot October 15, 2025 09:17
@Endkind Endkind added the maintenance Codebase cleanup or non-feature updates label Oct 15, 2025
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 maintenance PR refreshes the pre-commit configuration and standardizes setup scripts across Unix, Windows CMD, and PowerShell environments. The changes bump pre-commit tool versions and replace platform-specific setup scripts with improved versions that follow a unified flow.

  • Updates pre-commit tool versions (pre-commit-hooks v6.0.0, black 25.9.0, isort 6.1.0)
  • Replaces setup.bat with setup.cmd and adds new setup.ps1 for PowerShell support
  • Hardens setup.sh with stricter error handling and improves messaging consistency

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
.pre-commit-config.yaml Updates tool versions for pre-commit hooks, black, and isort
setup.sh Adds stricter error handling, improved messaging, and standardized flow
setup.ps1 New PowerShell setup script with error handling and consistent messaging
setup.cmd New CMD setup script replacing setup.bat with standardized flow
setup.bat Removed old Windows batch file

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

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings October 15, 2025 09:19
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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 4 out of 4 changed files in this pull request and generated no new comments.


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

@Endkind Endkind merged commit 6970556 into master Oct 15, 2025
1 check passed
@Endkind Endkind deleted the maintenance/update-pre-commit-setup-scripts-end branch October 15, 2025 09:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Codebase cleanup or non-feature updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants