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

CI fixes: update precommit | lock grpcio, protobuf versions #259

Merged
merged 8 commits into from
Nov 12, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 7 additions & 3 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ repos:
hooks:
- id: black
- repo: https://github.com/pre-commit/pre-commit-hooks
rev: v4.1.0
rev: v5.0.0
hooks:
- id: trailing-whitespace
- id: end-of-file-fixer
Expand Down Expand Up @@ -88,8 +88,12 @@ repos:
- id: ruff
exclude: (thirdparty|cpp/sophus)/.*$
args: [--fix, --exit-non-zero-on-fix]
- repo: https://github.com/PyCQA/docformatter
rev: v1.7.5

# TODO: Set ``rev: v1.7.6`` once the release is available.
# See: https://github.com/PyCQA/docformatter/issues/293
- repo: https://github.com/myint/docformatter
# rev: v1.7.6
rev: "eb1df347edd128b30cd3368dddc3aa65edcfac38"
hooks:
- id: docformatter
args: [--in-place, --wrap-summaries=115, --wrap-descriptions=120]
5 changes: 4 additions & 1 deletion pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,9 @@
requires = [
"setuptools",
"farm-ng-package",
"pybind11"
"pybind11",
"wheel",
"grpcio==1.64.1",
"grpcio-tools==1.64.1",
]
build-backend = "setuptools.build_meta"
31 changes: 11 additions & 20 deletions setup.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -21,43 +21,34 @@ classifiers =

[options]
python_requires = >=3.8
setup_requires =
wheel
install_requires =
farm_ng_package
protobuf
grpcio
protobuf==5.27.2
grpcio==1.64.1
psutil
numpy

tests_require =
pytest
pytest-runner
pytest-mypy
pylint-protobuf
types-protobuf
anyio

test_suite = tests

package_dir =
= py
packages =
farm_ng
farm_ng.core

[options.extras_require]
dev =
test =
pytest
pytest-mypy
pytest-runner
pylint-protobuf
anyio
types-protobuf
grpcio-tools==1.64.1

dev =
%(test)s
pre-commit
mypy
types-protobuf
pylint
grpcio-tools
mypy-protobuf
pylint-protobuf
anyio

[mypy]
files = py/farm_ng, py/tests
Expand Down
Loading