Skip to content

Conversation

cclauss
Copy link
Collaborator

@cclauss cclauss commented Aug 25, 2025

% pre-commit autoupdate

[https://github.com/pre-commit/pre-commit-hooks] updating v5.0.0 -> v6.0.0
[https://github.com/MarcoGorelli/auto-walrus] already up to date!
[https://github.com/codespell-project/codespell] already up to date!
[https://github.com/astral-sh/ruff-pre-commit] updating v0.12.1 -> v0.12.10

% pre-commit run --all-files

@cclauss cclauss changed the title pre-commit autoupdate 2025-08-25 On macOS v15.x, nsss has 177 voices Aug 25, 2025
@cclauss cclauss requested a review from Copilot August 25, 2025 08:12
Copy link

@Copilot 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 updates the expected voice count for macOS v15.x from 176 to 177 voices in test assertions and comments, and updates pre-commit hooks to newer versions.

  • Updates voice count expectations in test files to reflect the correct number of voices (177) available on macOS v15.x
  • Updates pre-commit hook versions for pre-commit-hooks (v5.0.0 → v6.0.0) and ruff-pre-commit (v0.12.1 → v0.12.10)

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
tests/test_pyttsx3.py Updates voice count from 176 to 177 in comments and assertions for macOS v15.x
tests/test_engines.py Updates voice count from 176 to 177 in comments and assertions for macOS v15.x
.pre-commit-config.yaml Updates pre-commit hook versions to latest releases

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

@cclauss cclauss merged commit 595ff64 into nateshmbhat:master Aug 25, 2025
12 checks passed
@cclauss cclauss deleted the pre-commit-autoupdate-2025-08-25 branch August 25, 2025 08:13
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.

1 participant