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

Drop Python 3.7 supports in Develop #3060

Merged
merged 16 commits into from
Sep 27, 2023
Merged

Conversation

noklam
Copy link
Contributor

@noklam noklam commented Sep 21, 2023

NOTE: Kedro datasets are moving from kedro.extras.datasets to a separate kedro-datasets package in
kedro-plugins repository. Any changes to the dataset implementations
should be done by opening a pull request in that repository.

Description

Close #2158

Development notes

  • Drop importlib_metadata, replace with standard library importlib.metadata

Checklist

  • Read the contributing guidelines
  • Signed off each commit with a Developer Certificate of Origin (DCO)
  • Opened this PR as a 'Draft Pull Request' if it is work-in-progress
  • Updated the documentation to reflect the code changes
  • Added a description of this change in the RELEASE.md file
  • Added tests to cover my changes
  • Checked if this change will affect Kedro-Viz, and if so, communicated that with the Viz team

Signed-off-by: Nok <nok.lam.chan@quantumblack.com>
Signed-off-by: Nok <nok.lam.chan@quantumblack.com>
Signed-off-by: Nok <nok.lam.chan@quantumblack.com>
@noklam noklam self-assigned this Sep 21, 2023
Signed-off-by: Nok <nok.lam.chan@quantumblack.com>
@astrojuanlu
Copy link
Member

You might want to try pyupgrade --py38-plus https://github.com/asottile/pyupgrade

@noklam
Copy link
Contributor Author

noklam commented Sep 21, 2023

Didn't know this exist, will try.

@noklam noklam marked this pull request as ready for review September 22, 2023 17:58
"importlib-metadata>=3.6,<7.0; python_version >= '3.8'",
"importlib_resources>=1.3,<7.0", # The `files()` API was introduced in `importlib_resources` 1.3 and Python 3.9.
"jmespath>=0.9.5",
"more_itertools>=8.14.0",
"omegaconf>=2.1.1",
"parse>=1.19.0",
"pip-tools>=6.5",
"pluggy>=1.0,<1.3", # TODO: Uncap when dropping Python 3.7 support, see https://github.com/kedro-org/kedro/issues/2979
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This causes nModuleNotFoundError: No module named \'pluggy\'

@noklam noklam marked this pull request as draft September 25, 2023 10:45
@@ -327,10 +327,10 @@ jobs:
- win_setup:
python_version: <<parameters.python_version>>
- when:
# Save Python package cache only for Python 3.7. The conda environment itself
# Save Python package cache only for Python 3.8. The conda environment itself
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Doesn't really matter since we are going to remove CCI

@noklam noklam requested review from astrojuanlu and SajidAlamQB and removed request for yetudada and merelcht September 27, 2023 13:29
@noklam noklam marked this pull request as ready for review September 27, 2023 13:29
Copy link
Contributor

@SajidAlamQB SajidAlamQB left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 🚀

Copy link
Member

@merelcht merelcht left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just one question about the pluggy bound, but otherwise looks good! Don't forget to add this to the release notes ✍️

@noklam noklam enabled auto-merge (squash) September 27, 2023 14:21
…kedro-org/kedro into noklam/drop-python-support-3058

Signed-off-by: Nok <nok.lam.chan@quantumblack.com>
Signed-off-by: Nok <nok.lam.chan@quantumblack.com>
@noklam noklam merged commit 02b101b into develop Sep 27, 2023
@noklam noklam deleted the noklam/drop-python-support-3058 branch September 27, 2023 14:56
@astrojuanlu
Copy link
Member

Farewell, Python 3.7 👋🏽

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.

4 participants