Maybe port all of the terrible c++ linters I've written in the past #21

Activity

1. asottile

   added

   enhancement

   on Mar 13, 2014
2. asottile commented on Jun 15, 2014

   

   asottile

   on Jun 15, 2014

   MemberAuthor

   More actions

   Most of these are grep based. I took a stab a quick stab at a few of them:

   -   id: space-after-keywords
       name: Space after keywords
       language: system
       entry: grep -E -H -n '^[ ]*(if|for|while|catch)\('
       files: \.(c|cc|cpp|h|hpp)$
       expected_return_value: 123
   -   id: space-after-comments
       name: Space after comments
       language: system
       entry: grep -E -H -n '(^| )//[^ ]'
       files: \.(cc|cpp|h|hpp)$
       expected_return_value: 123
   -   id: space-brace
       name: Space between paren and brace
       language: system
       entry: grep -H -n '){{'
       files: \.(c|cc|cpp|h|hpp)$
       expected_return_value: 123
   -   id: space-after-paren
       name: No spaces after parens
       language: system
       entry: grep -E -H -n '\( '
       files: \.(c|cc|cpp|h|hpp)$
       expected_return_value: 123
   -   id: include-space
       name: 'Needs space after #include'
       language: system
       entry: grep -E -H -n '^[ ]*#include[^ ]'
       files: \.(c|cc|cpp|h|hpp)$
       expected_return_value: 123
3. asottile commented on Jun 17, 2014

   

   asottile

   on Jun 17, 2014

   MemberAuthor

   More actions

   The above hooks don't actually work (I added a testcase demonstrating their problem to the pcre branch #120 . I'll rewrite these as pcre hooks later.
4. asottile

   closed this as completedon Jan 13, 2015
5. droctothorpe

   added 3 commits that reference this issue on Mar 23, 2022

   Add check_case_conflict hook pre-commit#21

   c682b50

   Add check-case-conflict to README.md pre-commit#21 pre-commit#22

   566b64a

   Merge pull request pre-commit#25 from guykisel/update-readme-case-con…

   ...

   5541a6a
6. goodspark

   added a commit that references this issue on Apr 2, 2025

   [clyde] auto-update venv if necessary (pre-commit#21)

   ...

   1867684