[misc] Update pre-commit hooks

[pre-commit-ci]

updates:

• github.com/google/yapf: v0.31.0 → v0.32.0
• https://github.com/timothycrosley/isort → https://github.com/PyCQA/isort
• github.com/pre-commit/mirrors-clang-format: v10.0.1 → v13.0.1
• github.com/pre-commit/pre-commit-hooks: v4.0.1 → v4.1.0
• github.com/PyCQA/pylint: v2.13.1 → v2.13.4

[netlify]

✅ Deploy Preview for docsite-preview canceled.

Name	Link
🔨 Latest commit	e22ebc2
🔍 Latest deploy log	https://app.netlify.com/sites/docsite-preview/deploys/6262334003a382000941c519

[frostming]

@qiao-bo I've added a rule for yapf, now the linter status turns green. It seems clang-format 12+ also removed a space in between &&... args, can we just accept and follow it?

[qiao-bo]

@qiao-bo I've added a rule for yapf, now the linter status turns green. It seems clang-format 12+ also added a space in between &&... args, can we just accept and follow it?

Thanks for the quick fix. About the added space: Is it possible to enforce this rule in our .clang-format file? (did we even use this config file? ;/) Many of us are still using clang 10 toolings and better to be consistent across commits.

[frostming]

This is a change in clang-format 11 and unfortunately, it doesn't expose a config for this.

Reference: https://stackoverflow.com/questions/65964384/how-to-make-clang-format-11-put-a-space-after-variadic-template-decleration

BTW, in fact, the new behavior is to remove the space, not add.

[qiao-bo]

This is a change in clang-format 11 and unfortunately, it doesn't expose a config for this.

Reference: https://stackoverflow.com/questions/65964384/how-to-make-clang-format-11-put-a-space-after-variadic-template-decleration

BTW, in fact, the new behavior is to remove the space, not add.

ok, then let's just accept this.

[ailzhang]

@frostming Is there a way to pin the yapf and clang-format version in pre-commit-bot instead?:P

[frostming]

@ailzhang I am afraid not, the PR will be sent each quarter. I have filed an FR to pre-commit/pre-commit#2331

[qiao-bo]

once per quarter should be fine. @ailzhang let's merge this?

[ailzhang]

LGTM!