[CI] Maybe redundant 'entry' key in .pre-commit-config.yaml

[asfimport]

Hi, occasionally I find a mior issue in the '.pre-commit-config.yaml' file that.

- id: cmake-format
    name: CMake Format
    language: python
    entry: bash -c "pip install cmake-format && python run-cmake-format.py --check"
    entry: echo
    files: ^(.**/CMakeLists.txt|.**.cmake)$
  

Maybe the item entry: echo is redundant. 

 

Reporter: Chen / @dota17

_{Note: This issue was originally created as ARROW-9426. Please see the migration documentation for further details.}

[asfimport]

FredGan / @fredgan:
Sorry that it's not redundant. But two same "entry" key in the same level of YAML maybe showed errror in some IDE, such as "Goland".

I think maybe it's better that merge the two entries to one, such like 

entry: bash -c "pip install cmake-format && python run-cmake-format.py --check; echo"

I run the "pre-commit run -a" , but I find many CMake files are re-formatted. 

How do you think about this? @wesm

 

[pitrou]

This issue is obsolete, closing.