Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GHA: Fix lint cache #322

Merged
merged 3 commits into from Mar 4, 2020
Merged

GHA: Fix lint cache #322

merged 3 commits into from Mar 4, 2020

Conversation

hugovk
Copy link
Member

@hugovk hugovk commented Mar 4, 2020

For some reason, when the last PR was merged, the lint failed because of an apparently outdated cache:

Run tox -e lint
lint create: /home/runner/work/pylast/pylast/.tox/lint
lint installdeps: pre-commit
lint installed: appdirs==1.4.3,cfgv==3.1.0,distlib==0.3.0,filelock==3.0.12,identify==1.4.11,nodeenv==1.3.5,pre-commit==2.1.1,PyYAML==5.3,six==1.14.0,toml==0.10.0,virtualenv==20.0.8
lint run-test-pre: PYTHONHASHSEED='3856264503'
lint run-test: commands[0] | pre-commit run --all-files
pyupgrade................................................................Failed
- hook id: pyupgrade
- exit code: 1

Executable `/home/runner/.cache/pre-commit/repoy32i7vy1/py_env-python3.8/bin/python` not found

black....................................................................Failed
- hook id: black
- exit code: 1

Executable `/home/runner/.cache/pre-commit/repo4hkcvbl4/py_env-python3/bin/python` not found

flake8...................................................................Failed
- hook id: flake8
- exit code: 1

Executable `/home/runner/.cache/pre-commit/repo0nx3ym4i/py_env-python3.8/bin/python` not found

seed isort known_third_party.............................................Failed
- hook id: seed-isort-config
- exit code: 1

There's no way (yet) to delete caches on GHA, you need to stick a version number in the key instead.

@hugovk hugovk added the changelog: skip Exclude PR from release draft label Mar 4, 2020
@codecov
Copy link

codecov bot commented Mar 4, 2020

Codecov Report

Merging #322 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #322   +/-   ##
=======================================
  Coverage   93.75%   93.75%           
=======================================
  Files           1        1           
  Lines        1265     1265           
=======================================
  Hits         1186     1186           
  Misses         79       79

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d367d91...7005dbc. Read the comment docs.

@hugovk hugovk merged commit 0f80585 into master Mar 4, 2020
@hugovk hugovk deleted the pre-commit-cache branch March 4, 2020 20:36
@hugovk hugovk mentioned this pull request Mar 6, 2020
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog: skip Exclude PR from release draft
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant