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

Version 3.4.1, fix missing dependencies #867

Merged
merged 3 commits into from
Sep 22, 2023

Conversation

ktbyers
Copy link
Collaborator

@ktbyers ktbyers commented Sep 22, 2023

Correct issue with broken typing_extensions (which was caused by earlier changes I made to pyproject.toml and missing tests where we should catch this type of issue).

@ktbyers
Copy link
Collaborator Author

ktbyers commented Sep 22, 2023

@nemith I reimplemented this as I wanted to remove some of the clutter (i.e. the isort changes).

I also re-added mypy_extensions as I wanted to keep the changes close to the minimum set needed to fix the issue with the broken release (and looking into Callback Protocols as the typing replacement for the Extended Callback Types was taking too long to research/understand properly).

I am totally open to removing mypy_extensions we should just look to do it as part of the next release.

@ktbyers ktbyers changed the title Fix release 3 4 0 Fix missing dependencies in Release 3.4.0 Sep 22, 2023
@ktbyers
Copy link
Collaborator Author

ktbyers commented Sep 22, 2023

I also tested that this imported properly from a poetry built wheel version of 3.4.1

# In blank/new virtual environment
$ pip install nornir-3.4.1-py3-none-any.whl

$ python
Python 3.11.0 (main, Nov 17 2022, 14:54:18) [GCC 7.3.1 20180712 (Red Hat 7.3.1-15)] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import nornir
>>> nornir.__version__
'3.4.1'

@ktbyers
Copy link
Collaborator Author

ktbyers commented Sep 22, 2023

@dbarrosop @ubaumann

FYI, I am going to push out a new Nornir release to fix the currently broken release. See issue ##861

This was caused by the pyproject.toml restructuring/fixes that I did earlier (i.e. imports that were not dev imports got categorized as dev imports) and that this step is not tested currently (i.e. we had no feedback from CI-CD system that it was broken...I will see if I can fix that later).

Thanks to @nemith for his reporting of the issue and doing a bunch of the work on the fix. We should at the removal proposal of mypy_extensions that he recommended in his other PR, but I didn't have time to research that tonight.

@ktbyers ktbyers changed the title Fix missing dependencies in Release 3.4.0 Version 3.4.1, fix missing dependencies Sep 22, 2023
@ktbyers ktbyers merged commit d8c547c into nornir-automation:main Sep 22, 2023
13 checks passed
@nemith
Copy link
Contributor

nemith commented Sep 22, 2023

Yeah these being only used for type checking it should be fairly low risk, but so should reoranging a pyproject.toml file so I understand.

For next minor releases how do you queue the changes? Are you ok for me resubmitting another PR to reapply removing mypy_extensions?

@ktbyers
Copy link
Collaborator Author

ktbyers commented Sep 22, 2023

@nemith Yes, definitely go ahead and resubmit the PR to remove mypy_extensions.

There is a good chance it will be a while before there is another minor release, but it is better to get this done in the near future when it is fresh in everyone's mind (or else it will probably just get put onto the back burner until the deprecated feature is actually removed).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants