pre-commit hook fails while running on pre-commit.ci

[billsioros]

An example run can be found here.

My pre-commit-config is the following:

ci:
  skip: [editorconfig-checker, export-requirements]
  autofix_commit_msg: 'refactor: `pre-commit.ci` auto fix'
  autofix_prs: true
  autoupdate_commit_msg: 'refactor: `pre-commit.ci` auto update'

repos:
  - repo: https://github.com/humitos/mirrors-autoflake.git
    rev: v1.1
    hooks:
      - id: autoflake
        args: ['--in-place', '--remove-all-unused-imports', '--remove-unused-variable']
  - repo: https://github.com/pycqa/isort
    rev: 5.9.1
    hooks:
      - id: isort
  - repo: https://github.com/psf/black
    rev: 21.6b0
    hooks:
      - id: black
        language_version: python3
  - repo: https://github.com/PyCQA/flake8
    rev: 3.9.2
    hooks:
      - id: flake8
        exclude: ^docs|examples|tests
        additional_dependencies: [wemake_python_styleguide]
  - repo: https://github.com/pre-commit/pre-commit-hooks
    rev: v4.0.1
    hooks:
      - id: trailing-whitespace
      - id: end-of-file-fixer
      - id: check-yaml
      - id: check-json
      - id: check-toml
      - id: debug-statements
  - repo: https://github.com/pre-commit/mirrors-mypy
    rev: v0.910
    hooks:
    - id: mypy
      args: [--config, .mypy.ini]
      files: ^dotify|tests/
      additional_dependencies: [types-all]
  - repo: https://github.com/commitizen-tools/commitizen
    rev: v2.17.11
    hooks:
      - id: commitizen
        stages: [commit-msg]
  - repo: https://github.com/editorconfig-checker/editorconfig-checker.python
    rev: 2.3.5
    hooks:
      - id: editorconfig-checker
  - repo: https://github.com/jendrikseipp/vulture
    rev: v2.3
    hooks:
      - id: vulture
  - repo: https://github.com/tox-dev/tox-ini-fmt
    rev: 0.5.1
    hooks:
      - id: tox-ini-fmt
  - repo: local
    hooks:
      - id: export-requirements
        name: export-requirements
        language: system
        pass_filenames: false
        entry: poetry export --without-hashes -o requirements.txt
        files: ^(pyproject.toml|poetry.lock)$

Thanks in advance!

[mmicu]

Hi. Could you try to run the program with sudo? The script downloads the executable and the location must be writable.

If it works with sudo, I can introduce a new variable for selecting the location where the executable will be downloaded.

[billsioros]

Hello @mmicu ,

I'm quite sure that there is no way to run the command with sudo on ci 😢 I'll open a corresponding issue on pre-commit.ci's end.

Feel free to check it out here.

[billsioros]

Hello again,

as you can see in the linked issue network is not available at runtime 🥳 . One possible solution could be to bundle the executable with the python package. Other than that I'm not sure what else could be done.

[mmicu]

Hi. Yes, I could attach the executable to the Python package. I will try to release a new version in a few days.

[billsioros]

@mmicu Hello again! Any news on this 😃 ?

[mmicu]

Hey. Probably I will have some time to work on it this week.

[mmicu]

Hi @billsioros. I added this kind of offline mode in the version 2.3.51. Could you please try it and let me know?

[billsioros]

Hi @mmicu, excuse me but I fail to notice a new release 😢

[mmicu]

Don't worry. Take your time to test it.

[billsioros]

Don't worry. Take your time to test it.

What I meant is that I cannot find a release with the tag 2.3.51 for example https://github.com/editorconfig-checker/editorconfig-checker.python/tree/2.3.51. Excuse me for not being clear enough.

[mmicu]

What I meant is that I cannot find a release with the tag 2.3.51 for example https://github.com/editorconfig-checker/editorconfig-checker.python/tree/2.3.51. Excuse me for not being clear enough.

No worries. It's in PyPI. You can install the latest version with pip install editorconfig-checker. Alternatively, you can use pip install . if you want to install it from the repo itself (offline-mode branch).

In any case, from the pre-commit-config that you posted in the first message, it will be enough to update the version from 2.3.5 to 2.3.51.

[billsioros]

What I meant is that I cannot find a release with the tag 2.3.51 for example https://github.com/editorconfig-checker/editorconfig-checker.python/tree/2.3.51. Excuse me for not being clear enough.

No worries. It's in PyPI. You can install the latest version with pip install editorconfig-checker. Alternatively, you can use pip install . if you want to install it from the repo itself (offline-mode branch).

In any case, from the pre-commit-config that you posted in the first message, it will be enough to update the version from 2.3.5 to 2.3.51.

Hello again,

The CI fails on build time. You can find the run here. I'm pretty sure pre-commit does not install the hooks via pip but via git clone, thus the following error is raised

cloning https://github.com/editorconfig-checker/editorconfig-checker.python:

fetch attempt 1...
    => error
    fatal: couldn't find remote ref 2.3.51
fetch attempt 2...
    => error
    fatal: couldn't find remote ref 2.3.51
fetch attempt 3...
    => error
    fatal: couldn't find remote ref 2.3.51

I believe a new release/tag must be created before even being able to test this out.

Edit: Forgot to include a link to the pre-commit-config.yaml, which I changed. 😋

[mmicu]

I created the tag 2.3.51. Could you please try to test it again?