Skip to content

Commit

Permalink
chore(dev-deps): remove poetry check pre-commit hook
Browse files Browse the repository at this point in the history 
There's a ideological divide that prevents poetry and pre-commit from
collaborating effectively to allow poetry pre-commit hooks to be updated
by pre-commit. This check isn't that useful so I'm removing it.
  • Loading branch information
@cpcloud
cpcloud committed Nov 8, 2022
1 parent abafe1b commit fc617e2
Showing 1 changed file with 0 additions and 5 deletions.
5 changes: 0 additions & 5 deletions .pre-commit-config.yaml
View file
Original file line number Diff line number Diff line change
Expand Up @@ -9,14 +9,9 @@ ci:
- just
- nixpkgs-fmt
- nix-linter
- poetry-check
default_stages:
- commit
repos:
- repo: https://github.com/python-poetry/poetry
rev: 1.2.2
hooks:
- id: poetry-check
- repo: https://github.com/pycqa/isort
rev: 5.10.1
hooks:
Expand Down

1 comment on commit fc617e2

@ibis-squawk-bot
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 3.

Benchmark suite Current: fc617e2 Previous: abafe1b Ratio
ibis/tests/benchmarks/test_benchmarks.py::test_large_expr_equals 7754.193895104283 iter/sec (stddev: 0.0031867315720536193) 275031.6802870731 iter/sec (stddev: 0.0000063850262867997386) 35.47

This comment was automatically generated by workflow using github-action-benchmark.

Please sign in to comment.