Skip to content

NiFi 1.26.0 update #359

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 3 commits into from
Closed

NiFi 1.26.0 update #359

wants to merge 3 commits into from

Conversation

michaelarnauts
Copy link
Contributor

  • Update the swagger definitions to 1.26.0 and re-ran the code generation

@michaelarnauts
Copy link
Contributor Author

I've tried to run the tests, but it seems they don't work on Python 3.12:

ImportError while importing test module '/home/michael/Development/nipyapi/tests/test_utils.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
/usr/lib/python3.12/importlib/__init__.py:90: in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
tests/test_utils.py:12: in <module>
    from deepdiff import DeepDiff
venv/lib/python3.12/site-packages/deepdiff/__init__.py:7: in <module>
    from .diff import DeepDiff
venv/lib/python3.12/site-packages/deepdiff/diff.py:19: in <module>
    from collections import Mapping
E   ImportError: cannot import name 'Mapping' from 'collections' (/usr/lib/python3.12/collections/__init__.py)

Also, I couldn't install all dev requirements since watchdog has to be an old version to support python 2.

I wonder if it's time to drop the Python 2 compatibility?

@Chaffelson
Copy link
Owner

Superceded by #365

@Chaffelson Chaffelson closed this Oct 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@michaelarnauts @Chaffelson @michaelarnauts-dstny