Skip to content

chore: enable the conventional-pre-commit run sucess and update to latest version#111

Merged
Cuiyus merged 3 commits intoalibaba:mainfrom
SYaoJun:214_fix_pre_commit
Feb 25, 2026
Merged

chore: enable the conventional-pre-commit run sucess and update to latest version#111
Cuiyus merged 3 commits intoalibaba:mainfrom
SYaoJun:214_fix_pre_commit

Conversation

@SYaoJun
Copy link
Contributor

@SYaoJun SYaoJun commented Feb 14, 2026

the original scripts is incorrect and not be triggered when commit a message.

Conventional Commit......................................................Failed
- hook id: conventional-pre-commit
- exit code: 1

usage: conventional-pre-commit [-h] [--force-scope] [--strict]
                               [types ...] input
conventional-pre-commit: error: unrecognized arguments: --types --scope-optional .git/COMMIT_EDITMSG

@SYaoJun
Copy link
Contributor Author

SYaoJun commented Feb 14, 2026

image

It seems Unit tests have some failed cases, Which can hardly be attributed to my changes. Do you know how to fix it? @Cuiyus

@Cuiyus
Copy link
Collaborator

Cuiyus commented Feb 25, 2026

Thank you for your submission!

@Cuiyus
Copy link
Collaborator

Cuiyus commented Feb 25, 2026

image It seems Unit tests have some failed cases, Which can hardly be attributed to my changes. Do you know how to fix it? @Cuiyus

Currently, there are some issues with the testing in C++ and they will be fixed by @iaojnh.

@Cuiyus Cuiyus merged commit 2b97fee into alibaba:main Feb 25, 2026
4 checks passed
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.

3 participants