Skip to content

change name from GLM-V to GLM-OCR#156

Closed
JaredforReal wants to merge 1 commit intomainfrom
name
Closed

change name from GLM-V to GLM-OCR#156
JaredforReal wants to merge 1 commit intomainfrom
name

Conversation

@JaredforReal
Copy link
Copy Markdown
Collaborator

Contribution Guide

We welcome your contributions to this repository. To ensure elegant code style and better code quality, we have prepared
the following contribution guidelines.

What We Accept

  • This PR fixes a typo or improves the documentation (if this is the case, you may skip the other checks).
  • This PR fixes a specific issue — please reference the issue number in the PR description. Make sure your code strictly
    follows the coding standards below.
  • This PR introduces a new feature — please clearly explain the necessity and implementation of the feature. Make sure
    your code strictly follows the coding standards below.

Code Style Guide

Good code style is an art. We have prepared a pre-commit hook to enforce consistent code
formatting across the project. You can clean up your code following the steps below:

pre-commit run --all-files

If your code complies with the standards, you should not see any errors.

Naming Conventions

  • Please use English for naming; do not use Pinyin or other languages. All comments should also be in English.
  • Follow PEP8 naming conventions strictly, and use underscores to separate words. Avoid meaningless names such as
    a, b, c.

Signed-off-by: JaredforReal <w13431838023@gmail.com>
Copilot AI review requested due to automatic review settings March 20, 2026 03:22
Copy link
Copy Markdown
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 PR updates the skill documentation to consistently use the GLM-OCR name (instead of GLM-V) in the skill titles, aligning the skills with the rest of the GLM-OCR repository branding.

Changes:

  • Rename “GLM-V … Skill” headers to “GLM-OCR … Skill” in three skill docs.
  • Keep the existing skill descriptions and metadata intact while correcting the displayed product name.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
skills/glmocr-table/SKILL.md Updates the skill title from GLM-V to GLM-OCR.
skills/glmocr-handwriting/SKILL.md Updates the skill title from GLM-V to GLM-OCR.
skills/glmocr-formula/SKILL.md Updates the skill title from GLM-V to GLM-OCR.

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

---

# GLM-V Table Recognition Skill / GLM-V 表格识别技能
# GLM-OCR Table Recognition Skill / GLM-OCR 表格识别技能
Copy link

Copilot AI Mar 20, 2026

Choose a reason for hiding this comment

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

The PR description currently contains what looks like a generic contribution guide template and doesn’t describe this change (renaming GLM-V to GLM-OCR in skill docs). Please update the PR description to briefly state the scope/intent of this PR so reviewers and release notes are accurate.

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.

2 participants