Skip to content

fix README#3

Open
njzjz wants to merge 1 commit intoallganize:mainfrom
njzjz:patch-1
Open

fix README#3
njzjz wants to merge 1 commit intoallganize:mainfrom
njzjz:patch-1

Conversation

@njzjz
Copy link

@njzjz njzjz commented Feb 1, 2026

Summary

fix README

Test Plan

Copilot AI review requested due to automatic review settings February 1, 2026 19:03
Copy link

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

Updates the README’s pre-commit configuration examples to use a cleaner, standard tag for the hook rev.

Changes:

  • Replaced the malformed/overlong rev: value with v0.0.14 in three README examples.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 21 to +24
repos:
- repo: https://github.com/allganize/ty-pre-commit
# Ty version.
rev: v0.0.14a35a34a33a32a31a30a29a28a27a26a25a24a23a22a21a20
rev: v0.0.14
Copy link

Copilot AI Feb 1, 2026

Choose a reason for hiding this comment

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

The pre-commit examples point to https://github.com/allganize/ty-pre-commit, but elsewhere in this README and repo (e.g., Actions badge uses astral-sh/ty-pre-commit, workflow release notes link to astral-sh/ty) it looks like the canonical repository is astral-sh/ty-pre-commit. This mismatch will cause users to copy an incorrect repo: URL. Please align the repo: URL in these examples with the actual repository (or update the badges/links if the project has moved).

Copilot uses AI. Check for mistakes.
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