Skip to content

chore: enhance pre-commit configuration with additional hooks for rob…#12

Merged
Rasic2 merged 2 commits intomasterfrom
dev/zhouh
Sep 14, 2025
Merged

chore: enhance pre-commit configuration with additional hooks for rob…#12
Rasic2 merged 2 commits intomasterfrom
dev/zhouh

Conversation

@Rasic2
Copy link
Owner

@Rasic2 Rasic2 commented Sep 14, 2025

…ustness and file integrity checks

Summary by CodeRabbit

  • Bug Fixes

    • Corrected sign errors in the Rodrigues rotation matrix in the math documentation.
    • Fixed release notes generation to properly exclude the “skip-changelog” label.
  • Documentation

    • Restored the Go language page to the navigation.
    • Cleaned up whitespace/newlines and minor formatting across multiple docs (no content changes).
  • Chores

    • Expanded and enabled comprehensive pre-commit hooks (linting, formatting, validators, security checks).
    • Normalized CI/workflow files with trailing newlines (no behavioral changes).

@coderabbitai
Copy link

coderabbitai bot commented Sep 14, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

Configuration and documentation updates: fix YAML indentation in Release Drafter, enable and expand pre-commit hooks, normalize whitespace/trailing newlines across workflows, docs, and configs, adjust minor doc formatting, and correct two signs in a Rodrigues rotation matrix in documentation.

Changes

Cohort / File(s) Summary of Changes
Release Drafter config fix
.github/release-drafter.yml
Corrected YAML list indentation under exclude-labels to a single-level list.
Workflow whitespace normalization
.github/workflows/check-pr-title.yml, .github/workflows/pr-auto-label.yml
Added trailing newlines and removed stray whitespace; no logic or behavior changes.
Pre-commit enablement and expansion
.pre-commit-config.yaml
Activated and expanded a broad set of pre-commit hooks; added pyupgrade hook; included targeted excludes; no runtime code changes.
ReadTheDocs and requirements formatting
.readthedocs.yaml, requirements.txt
Added trailing newlines; no content changes.
Sphinx conf formatting
docs/source/conf.py
Whitespace normalization; switched to single quotes in lists; added trailing newline; semantics unchanged.
Docs: development files formatting
docs/source/development/* (e.g., development.rst, MANIFEST.in, migration.md, mirror.rst, package.rst, version.md)
Trailing newline and whitespace fixes; no content changes.
Docs: toctrees and pages formatting
docs/source/game.rst, docs/source/language.rst, docs/source/language/python.rst, docs/source/others.rst
Trailing newline fixes; language/go.md entry reaffirmed; no structural changes otherwise.
Docs: math correction
docs/source/math.md
Fixed signs in two entries of the Rodrigues rotation matrix; documentation-only correction.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

I thump my paws at tidy lines,
Hooks now hop where logic shines.
YAML straight, the drafts aligned,
Matrices with signs refined.
With whiskers twitching, I commit—
Carrots, checks, and docs that fit. 🥕✨

✨ Finishing touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch dev/zhouh

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 63ac066 and 43a1315.

📒 Files selected for processing (18)
  • .github/release-drafter.yml (1 hunks)
  • .github/workflows/check-pr-title.yml (1 hunks)
  • .github/workflows/pr-auto-label.yml (2 hunks)
  • .pre-commit-config.yaml (1 hunks)
  • .readthedocs.yaml (1 hunks)
  • docs/source/conf.py (2 hunks)
  • docs/source/development.rst (1 hunks)
  • docs/source/development/MANIFEST.in (1 hunks)
  • docs/source/development/migration.md (1 hunks)
  • docs/source/development/mirror.rst (1 hunks)
  • docs/source/development/package.rst (1 hunks)
  • docs/source/development/version.md (1 hunks)
  • docs/source/game.rst (1 hunks)
  • docs/source/language.rst (1 hunks)
  • docs/source/language/python.rst (1 hunks)
  • docs/source/math.md (1 hunks)
  • docs/source/others.rst (1 hunks)
  • requirements.txt (1 hunks)

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions bot added the chore label Sep 14, 2025
@github-actions github-actions bot added chore and removed chore labels Sep 14, 2025
@Rasic2 Rasic2 merged commit 2b39282 into master Sep 14, 2025
7 checks passed
@github-actions github-actions bot added chore and removed chore labels Sep 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant