Skip to content

Commit

Permalink
ci: pre-commit autoupdate (#4897)
Browse files Browse the repository at this point in the history
* ci: pre-commit autoupdate [skip ci]

updates:
- [github.com/astral-sh/ruff-pre-commit: v0.5.0 → v0.5.6](astral-sh/ruff-pre-commit@v0.5.0...v0.5.6)
- [github.com/pdm-project/pdm: 2.16.1 → 2.17.3](pdm-project/pdm@2.16.1...2.17.3)
- [github.com/bufbuild/buf: v1.34.0 → v1.35.1](bufbuild/buf@v1.34.0...v1.35.1)

* chore: update check to use isinstance

Signed-off-by: Aaron Pham <[email protected]>

---------

Signed-off-by: Aaron Pham <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Aaron Pham <[email protected]>
  • Loading branch information
pre-commit-ci[bot] and aarnphm authored Aug 6, 2024
1 parent 7e1f10c commit bdaa41b
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
6 changes: 3 additions & 3 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ ci:
exclude: '(.*\.(css|js|svg))|(.*/(snippets|grpc|proto)/.*)$'
repos:
- repo: https://github.com/astral-sh/ruff-pre-commit
rev: 'v0.5.0'
rev: 'v0.5.6'
hooks:
- id: ruff
args: [--fix, --exit-non-zero-on-fix, --show-fixes]
Expand All @@ -22,11 +22,11 @@ repos:
- id: trailing-whitespace
- id: end-of-file-fixer
- repo: https://github.com/pdm-project/pdm
rev: 2.16.1
rev: 2.17.3
hooks:
- id: pdm-lock-check
- repo: https://github.com/bufbuild/buf
rev: v1.34.0
rev: v1.35.1
hooks:
- id: buf-format
args: [--config=src/bentoml/grpc/buf.yaml, src/bentoml/grpc]
Expand Down
2 changes: 1 addition & 1 deletion src/bentoml/_internal/frameworks/transformers.py
Original file line number Diff line number Diff line change
Expand Up @@ -1074,7 +1074,7 @@ def save_model(
assert "impl" in task_definition, "'task_definition' requires 'impl' key."

impl = task_definition["impl"]
if type(pipeline_) != impl:
if isinstance(pipeline_, impl):
raise BentoMLException(
f"Argument 'pipeline' is not an instance of {impl}. It is an instance of {type(pipeline_)}."
)
Expand Down

0 comments on commit bdaa41b

Please sign in to comment.