Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[lint] auto format all by pre-commit, including c++, python #243

Merged
merged 1 commit into from
Dec 7, 2023

Conversation

cdliang11
Copy link
Collaborator

@cdliang11 cdliang11 commented Dec 7, 2023

follow: wenet-e2e/wenet#2199
Automatic reformat all the old code by pre-commit run --all-files, and then we got

(base) ➜  wespeaker-update git:(chengdong-lint-all) ✗ git commit -m "[lint] auto format all by pre-commit, including c++, python"
trim trailing whitespace.................................................Passed
yapf.....................................................................Passed
flake8...................................................................Passed
clang-format.............................................................Passed
cpplint..................................................................Passed
[chengdong-lint-all 8f49658] [lint] auto format all by pre-commit, including c++, python
 Committer: user01 <[email protected]>
Your name and email address were configured automatically based
on your username and hostname. Please check that they are accurate.
You can suppress this message by setting them explicitly:

    git config --global user.name "Your Name"
    git config --global user.email [email protected]

After doing this, you may fix the identity used for this commit with:

    git commit --amend --reset-author

 71 files changed, 779 insertions(+), 542 deletions(-)
 create mode 100644 .clang-format

@robin1001 robin1001 merged commit fb2325c into master Dec 7, 2023
3 checks passed
@robin1001 robin1001 deleted the chengdong-lint-all branch December 7, 2023 02:54
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