Skip to content

Precommit (bash): Enable space-redirects in shfmt - #8276

Open
Growl1234 wants to merge 2 commits into
deepmodeling:developfrom
Growl1234:precommit-bash
Open

Growl1234 wants to merge 2 commits into
deepmodeling:developfrom
Growl1234:precommit-bash

Conversation

@Growl1234

Copy link
Copy Markdown
Collaborator

cat a >b -> cat a > b

I also enabled the error report of precommit CI. The clang-format section is currently commented until all precommit work is done.

@ABACUS-PR-BOT ABACUS-PR-BOT left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Static review of f0f7bcb2b found no implementation defect; one nonblocking rollout suggestion follows. No hooks, builds or runtime tests were run.

Comment thread .pre-commit-config.yaml Resolved
@mohanchen mohanchen added Refactor Refactor ABACUS codes Compile & CICD & Docs & Dependencies Issues related to compiling ABACUS Standards Coding standards and governance rules labels Nov 4, 2026
@Growl1234
Growl1234 force-pushed the precommit-bash branch 2 times, most recently from ec6a989 to 3dcf96e Compare November 8, 2026 22:06

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Compile & CICD & Docs & Dependencies Issues related to compiling ABACUS Refactor Refactor ABACUS codes Standards Coding standards and governance rules

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants