search

wemake-services / wemake-python-styleguide

The strictest and most opinionated python linter ever!
https://wemake-python-styleguide.rtfd.io
MIT License
2.55k stars 378 forks source link

reviewdog: PullRequest needs 'git' command: failed to run 'git rev-parse --show-prefix' #2460

Open anuj-scanova opened 2 years ago

anuj-scanova commented 2 years ago

What's wrong

Produces error in actions

Run wemake-services/wemake-python-styleguide@0.16.1
  with:
    reporter: github-pr-review
    path: ./src
  env:
    pythonLocation: /opt/hostedtoolcache/Python/3.9.13/x64
    LD_LIBRARY_PATH: /opt/hostedtoolcache/Python/3.9.13/x64/lib
    GITHUB_TOKEN: ***
/usr/bin/docker run --name cd98fe6f4b56db05[2](https://github.com/user/backend/runs/7426747689?check_suite_focus=true#step:5:2)4edd9b7[3](https://github.com/user/backend/runs/7426747689?check_suite_focus=true#step:5:3)a672f703ae26_9e185d --label [4](https://github.com/user/backend/runs/7426747689?check_suite_focus=true#step:5:4)cd98f --workdir /github/workspace --rm -e pythonLocation -e LD_LIBRARY_PATH -e GITHUB_TOKEN -e INPUT_REPORTER -e INPUT_PATH -e HOME -e GITHUB_JOB -e GITHUB_REF -e GITHUB_SHA -e GITHUB_REPOSITORY -e GITHUB_REPOSITORY_OWNER -e GITHUB_RUN_ID -e GITHUB_RUN_NUMBER -e GITHUB_RETENTION_DAYS -e GITHUB_RUN_ATTEMPT -e GITHUB_ACTOR -e GITHUB_WORKFLOW -e GITHUB_HEAD_REF -e GITHUB_BASE_REF -e GITHUB_EVENT_NAME -e GITHUB_SERVER_URL -e GITHUB_API_URL -e GITHUB_GRAPHQL_URL -e GITHUB_REF_NAME -e GITHUB_REF_PROTECTED -e GITHUB_REF_TYPE -e GITHUB_WORKSPACE -e GITHUB_ACTION -e GITHUB_EVENT_PATH -e GITHUB_ACTION_REPOSITORY -e GITHUB_ACTION_REF -e GITHUB_PATH -e GITHUB_ENV -e GITHUB_STEP_SUMMARY -e RUNNER_OS -e RUNNER_ARCH -e RUNNER_NAME -e RUNNER_TOOL_CACHE -e RUNNER_TEMP -e RUNNER_WORKSPACE -e ACTIONS_RUNTIME_URL -e ACTIONS_RUNTIME_TOKEN -e ACTIONS_CACHE_URL -e GITHUB_ACTIONS=true -e CI=true -v "/var/run/docker.sock":"/var/run/docker.sock" -v "/home/runner/work/_temp/_github_home":"/github/home" -v "/home/runner/work/_temp/_github_workflow":"/github/workflow" -v "/home/runner/work/_temp/_runner_file_commands":"/github/file_commands" -v "/home/runner/work/qcg-backend/qcg-backend":"/github/workspace" 4cd98f:e6f4b[5](https://github.com/user/backend/runs/7426747689?check_suite_focus=true#step:5:5)[6](https://github.com/user/backend/runs/7426747689?check_suite_focus=true#step:5:6)db0524edd9b[7](https://github.com/user/backend/runs/7426747689?check_suite_focus=true#step:5:7)3a672f703ae26  "./src" "github-pr-review"
Using reporter: github-pr-review
Linting path: ./src
flake[8](https://github.com/user/backend/runs/7426747689?check_suite_focus=true#step:5:8) --version:
4.0.1 (flake8-bandit: 3.0.0, flake8-broken-line: 0.4.0, flake8-bugbear:
22.7.1, flake8-comprehensions: 3.10.0, flake8-darglint: 1.8.1,
flake8-debugger: 4.1.2, flake8-docstrings: 1.6.0, pydocstyle: 6.1.1,
flake8-eradicate: 1.2.1, flake8-string-format: 0.3.0, flake8_commas: 2.1.0,
flake8_isort: 4.1.1, flake8_quotes: 3.3.1, mccabe: 0.6.1, naming: 0.12.1,
pycodestyle: 2.8.0, pyflakes: 2.4.0, rst-docstrings: 0.2.7,
wemake_python_styleguide: 0.16.1) CPython 3.[9](https://github.com/user/backend/runs/7426747689?check_suite_focus=true#step:5:10).[10](https://github.com/user/backend/runs/7426747689?check_suite_focus=true#step:5:11) on Linux
=================================
multiprocessing.pool.RemoteTraceback: 
"""
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/multiprocessing/pool.py", line [12](https://github.com/user/backend/runs/7426747689?check_suite_focus=true#step:5:13)5, in worker
    result = (True, func(*args, **kwds))
  File "/usr/local/lib/python3.9/multiprocessing/pool.py", line 48, in mapstar
    return list(map(*args))
  File "/usr/local/lib/python3.9/site-packages/flake8/checker.py", line 687, in _run_checks
    return checker.run_checks()
  File "/usr/local/lib/python3.9/site-packages/flake8/checker.py", line 597, in run_checks
    self.run_ast_checks()
  File "/usr/local/lib/python3.9/site-packages/flake8/checker.py", line 500, in run_ast_checks
    for (line_number, offset, text, _) in runner:
  File "/usr/local/lib/python3.9/site-packages/flake8_docstrings.py", line [14](https://github.com/user/backend/runs/7426747689?check_suite_focus=true#step:5:15)2, in run
    for error in self._check_source():
  File "/usr/local/lib/python3.9/site-packages/flake8_docstrings.py", line 127, in _check_source
    for err in self._call_check_source():
  File "/usr/local/lib/python3.9/site-packages/pydocstyle/checker.py", line 149, in check_source
    error = this_check(
  File "/usr/local/lib/python3.9/site-packages/pydocstyle/checker.py", line [19](https://github.com/user/backend/runs/7426747689?check_suite_focus=true#step:5:20)9, in check_docstring_missing
    and is_blank(ast.literal_eval(docstring))
  File "/usr/local/lib/python3.9/ast.py", line 105, in literal_eval
    return _convert(node_or_string)
  File "/usr/local/lib/python3.9/ast.py", line 104, in _convert
    return _convert_signed_num(node)
  File "/usr/local/lib/python3.9/ast.py", line 78, in _convert_signed_num
    return _convert_num(node)
  File "/usr/local/lib/python3.9/ast.py", line 69, in _convert_num
    _raise_malformed_node(node)
  File "/usr/local/lib/python3.9/ast.py", line 66, in _raise_malformed_node
    raise ValueError(f'malformed node or string: {node!r}')
ValueError: malformed node or string: <ast.JoinedStr object at 0x7fc3e569a730>
"""
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
  File "/usr/local/bin/flake8", line 8, in <module>
    sys.exit(main())
  File "/usr/local/lib/python3.9/site-packages/flake8/main/cli.py", line [22](https://github.com/user/backend/runs/7426747689?check_suite_focus=true#step:5:23), in main
    app.run(argv)
  File "/usr/local/lib/python3.9/site-packages/flake8/main/application.py", line 375, in run
    self._run(argv)
  File "/usr/local/lib/python3.9/site-packages/flake8/main/application.py", line 364, in _run
    self.run_checks()
  File "/usr/local/lib/python3.9/site-packages/flake8/main/application.py", line [27](https://github.com/user/backend/runs/7426747689?check_suite_focus=true#step:5:28)1, in run_checks
    self.file_checker_manager.run()
  File "/usr/local/lib/python3.9/site-packages/flake8/checker.py", line 309, in run
    self.run_parallel()
  File "/usr/local/lib/python3.9/site-packages/flake8/checker.py", line 275, in run_parallel
    for ret in pool_map:
  File "/usr/local/lib/python3.9/multiprocessing/pool.py", line 448, in <genexpr>
    return (item for chunk in result for item in chunk)
  File "/usr/local/lib/python3.9/multiprocessing/pool.py", line 870, in next
    raise value
ValueError: malformed node or string: <ast.JoinedStr object at 0x7fc3e569a730>
reviewdog: PullRequest needs 'git' command: failed to run 'git rev-parse --show-prefix': exit status 1[28](https://github.com/user/backend/runs/7426747689?check_suite_focus=true#step:5:29)
=================================

Also, the actions are passing successfully despite the error.

Screen Shot 2022-07-20 at 3 48 14 PM

How it should be

The pipeline should run successfully and the action should fail in case of error.

Flake8 version and plugins

actions.py

name: Code Review

on:
  pull_request:
    branches:
      - qa
      - master

jobs:
  code-review:
    name: Code Review
    runs-on: ubuntu-latest
    steps:
      - name: Checkout
        uses: actions/checkout@v3
      - name: Setup python
        uses: actions/setup-python@v2
        with:
          python-version: 3.9
      - name: Check style guide
        uses: wemake-services/wemake-python-styleguide
        with:
          reporter: github-pr-review
          path: './src'
        env:
          GITHUB_TOKEN: ${{ secrets.github_token }}

pip information

Python 3.9

OS information

ubuntu-latest

sobolevn commented 2 years ago

Does your code pass flake8 . on the same python version?

anuj-scanova commented 2 years ago

@sobolevn Python version used Python 3.9.11

when I run the command flake8 . locally, there are few changes recommended without any error 

user@aa30b05174d3:/app$ flake8 .
./authentication/tasks.py:6:1: F401 'dateutil.relativedelta.relativedelta' imported but unused
./authentication/tasks.py:13:1: F401 'authentication.emails.send_inactive_account_deletion_reminder' imported but unused
./authentication/tasks.py:13:1: F401 'authentication.emails.send_inactive_account_deletion_confirmation_mail' imported but unused
./authentication/tasks.py:299:121: E501 line too long (122 > 120 characters)
./authentication/tasks.py:300:121: E501 line too long (121 > 120 characters)
./qcg/settings/__init__.py:12:4: F405 'DEBUG' may be undefined, or defined from star imports: .base, .celery, .constants, .ecs, .logging, .mfa, .monitoring, .payment, .security, .testing
./qcg/settings/__init__.py:12:14: F405 'ENVIRONMENT' may be undefined, or defined from star imports: .base, .celery, .constants, .ecs, .logging, .mfa, .monitoring, .payment, .security, .testing
./scan_data_collect/tasks.py:91:29: F541 f-string is missing placeholders
./tracking/google_tracking2.py:1:1: F401 'json' imported but unused
sobolevn commented 2 years ago

Are you sure that there are no extra files in the CI?

anuj-scanova commented 2 years ago

@sobolevn I listed the files in the actions and here is the output

Run ls -la
total 26
drwxr-xr-x 11 runner docker   4096 Jul 20 15:27 .
drwxr-xr-x  3 runner docker   4096 Jul 20 15:27 ..
-rw-r--r--  1 runner docker    149 Jul 20 15:27 .coveragerc
-rw-r--r--  1 runner docker    24 Jul 20 15:27 .dockerignore
drwxr-xr-x  8 runner docker   406 Jul 20 15:27 .git
drwxr-xr-x  3 runner docker   4096 Jul 20 15:27 .github
-rw-r--r--  1 runner docker    915 Jul 20 15:27 .gitignore
-rw-r--r--  1 runner docker   4352 Jul 20 15:27 Dockerfile
-rw-r--r--  1 runner docker   4340 Jul 20 15:27 Dockerfile.dev
-rw-r--r--  1 runner docker   1373 Jul 20 15:27 Pipfile.bk
-rw-r--r--  1 runner docker 135592 Jul 20 15:27 Pipfile.lock.bk
-rw-r--r--  1 runner docker    750 Jul 20 15:27 appspec-qa.yml
-rw-r--r--  1 runner docker    765 Jul 20 15:27 appspec-staging.yml
-rw-r--r--  1 runner docker    671 Jul 20 15:27 appspec-v8.yml
-rw-r--r--  1 runner docker    670 Jul 20 15:27 appspec.yml
-rw-r--r--  1 runner docker   6908 Jul 20 15:27 bitbucket-pipelines.yml
-rw-r--r--  1 runner docker   4180 Jul 20 15:27 codedeploy_deploy.py
drwxr-xr-x  6 runner docker   4096 Jul 20 15:27 configs
-rw-r--r--  1 runner docker   2774 Jul 20 15:27 docker-compose.dev.yml
-rw-r--r--  1 runner docker   1656 Jul 20 15:27 docker-compose.qa.yml
-rw-r--r--  1 runner docker   2370 Jul 20 15:27 docker-compose.staging.yml
-rw-r--r--  1 runner docker   1658 Jul 20 15:27 docker-compose.v8.yml
-rw-r--r--  1 runner docker   3217 Jul 20 15:27 docker-compose.yml
drwxr-xr-x  3 runner docker   4096 Jul 20 15:27 docs
drwxr-xr-x  3 runner docker   4096 Jul 20 15:27 library_data
-rw-r--r--  1 runner docker    989 Jul 20 15:27 pytest.ini
-rw-r--r--  1 runner docker   2331 Jul 20 15:27 readme.md
drwxr-xr-x  2 runner docker   4096 Jul 20 15:27 requirements
drwxr-xr-x  6 runner docker   4096 Jul 20 15:27 scripts
drwxr-xr-x 29 runner docker   4096 Jul 20 15:27 src
drwxr-xr-x  2 runner docker   4096 Jul 20 15:27 static_my_project
-rwxr-xr-x  1 runner docker   49 Jul 20 15:27 update_ecs_service.sh

And I have to check the src directory.

Also, the python version in actions is 3.9.13

Also, there is .flake8 file inside the src directory with the following content

[flake8]
exclude = .git,*migrations*,*tests*,*src/migration/*
max-line-length = 120
sobolevn commented 2 years ago

Can you please create a reproducable sample?