You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
fix: start fix of issue 694 with more pythonic __all__ array. - #723
This replaces #722, which is good. But no-op aliases might be confusing for future maintainers and the more "pythonic" approach is to use the documented __all__ variable.
Note that ac1982's tests from that PR are gratefully being reused here.
addendum
Update the examples/asynchronous_batching.py example to handle potential timeout errors on write.
✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.53%. Comparing base (ef60f89) to head (f2465a5).
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.
This PR addresses #694 by explicitly exporting the package’s public API through __all__, replacing the alias-based approach in #722 to support strict type checkers.
Changes:
Add an explicit public export list and a changelog entry.
Add runtime and mypy regression checks, with mypy included in test dependencies.
Update the asynchronous batching example’s timeout and error propagation.
File
Description
tests/typing/public_exports.py
Exercise representative public imports and runtime behavior.
tests/test_PublicExports.py
Run the import fixture directly and through mypy.
setup.py
Add mypy to test dependencies.
influxdb_client/__init__.py
Declare public exports using __all__.
examples/asynchronous_batching.py
Increase the timeout and propagate write errors to the completion future.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #694
Proposed Changes
Add the
__all__array to__init__.pyThis replaces #722, which is good. But no-op aliases might be confusing for future maintainers and the more "pythonic" approach is to use the documented
__all__variable.Note that ac1982's tests from that PR are gratefully being reused here.
addendum
Update the
examples/asynchronous_batching.pyexample to handle potential timeout errors on write.Checklist
pytest testscompletes successfully