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

Set minimum version of typing_extensions to 4.0.0 #759

Merged
merged 2 commits into from
Oct 17, 2023
Merged

Conversation

thaiphv
Copy link
Contributor

@thaiphv thaiphv commented Oct 17, 2023

typing_extensions.Self was introduced in 4.0.0 so making it as a minimum version required.

@thaiphv
Copy link
Contributor Author

thaiphv commented Oct 17, 2023

@vgrem the tests failed. Perhaps you haven't updated the secret OFFICE365_PYTHON_SDK_SECUREVARS. Can you please have a look?

@vgrem
Copy link
Owner

vgrem commented Oct 17, 2023

@thaiphv, thank you for the improvements! Regarding failed GitHub pipeline, indeed, in addition to expired credentials a few more errors needs to be addressed, among them:

ImportError while importing test module '/home/runner/work/Office365-REST-Python-Client/Office365-REST-Python-Client/test*//.py'.
Hint: make sure your test modules/packages have valid Python names.

ModuleNotFoundError: No module named 'typing_extensions'

@vgrem vgrem merged commit d52ee52 into vgrem:master Oct 17, 2023
1 check failed
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.

None yet

2 participants