diff --git a/python/instrumentation/openinference-instrumentation-openai/src/openinference/instrumentation/openai/_request.py b/python/instrumentation/openinference-instrumentation-openai/src/openinference/instrumentation/openai/_request.py index 7e16a6ae1..e1f1a1f88 100644 --- a/python/instrumentation/openinference-instrumentation-openai/src/openinference/instrumentation/openai/_request.py +++ b/python/instrumentation/openinference-instrumentation-openai/src/openinference/instrumentation/openai/_request.py @@ -177,9 +177,10 @@ def _finalize_response( if ( self._is_streaming(response) or hasattr( - # FIXME: Ideally we should not rely on a private attribute (but it may be impossible). - # The assumption here is that calling `.parse()` stores the stream object in `._parsed` - # and calling `.parse()` again will not overwrite the monkey-patched version. + # FIXME: Ideally we shouldn't rely on a private attribute (but it may be + # impossible). The assumption here is that calling `.parse()` stores the + # stream object in `._parsed` and calling `.parse()` again will not + # overwrite the monkey-patched version. # See https://github.com/openai/openai-python/blob/f1c7d714914e3321ca2e72839fe2d132a8646e7f/src/openai/_response.py#L65 # noqa: E501 response, "_parsed", diff --git a/python/tox.ini b/python/tox.ini index cb9edf390..4136a7a3d 100644 --- a/python/tox.ini +++ b/python/tox.ini @@ -23,6 +23,6 @@ commands = mypy: mypy --config-file {toxinidir}/mypy.ini --explicit-package-bases {posargs:src} test: pytest {posargs:tests} ci: ruff format . --diff --config {toxinidir}/ruff.toml - ci: ruff check . --diff --config {toxinidir}/ruff.toml + ci: ruff check . --no-fix --config {toxinidir}/ruff.toml ci: mypy --config-file {toxinidir}/mypy.ini --explicit-package-bases src ci: pytest tests