Skip to content

Conversation

@tony
Copy link
Contributor

@tony tony commented Jan 18, 2022

Resolves: #4997

Pull Request Check List

Resolves: #4997

  • Added tests for changed code.
  • Updated documentation for changed code.

@neersighted
Copy link
Member

There are a couple more things to do here, I think:

  • the Python version in .flake8
  • the arguments to pyupgrade in .pre-commit-config.yaml

@tony tony marked this pull request as draft January 18, 2022 14:31
@tony
Copy link
Contributor Author

tony commented Jan 18, 2022

There are a couple more things to do here, I think:

  • the Python version in .flake8
  • the arguments to pyupgrade in .pre-commit-config.yaml

I will convert it to a draft, handle those things, and also continue to look for any other places

If you notice anything else let me know

@tony
Copy link
Contributor Author

tony commented Jan 18, 2022

pre-commit.ci autofix

@neersighted
Copy link
Member

neersighted commented Jan 18, 2022

Looks like we can drop the dataclasses dep as the marker (python = "<3.7") makes it a no-op.

I don't have time to look into this, but I also suspect this line can be dropped as the types should be in the stdlib now. I may be wrong, so please look into it before making that change.

@tony tony dismissed a stale review via 7c9703f January 20, 2022 10:56
@tony
Copy link
Contributor Author

tony commented Jan 20, 2022

@neersighted

@tony tony marked this pull request as ready for review January 22, 2022 17:12
@tony tony requested a review from branchv January 22, 2022 17:12
@tony tony force-pushed the deprecate-python-3.6 branch from aec2dc9 to 5841dc0 Compare January 22, 2022 17:14
@tony
Copy link
Contributor Author

tony commented Jan 22, 2022

@branchvincent: Thank you

Rebased.

Can you approve running on python-poetry/poetry-core#263?

@tony
Copy link
Contributor Author

tony commented Jan 22, 2022

@branchvincent

P.S.
image

Any preference on how to navigate this? I can merge the commits into mine or re-do them myself.

@branchv
Copy link
Member

branchv commented Jan 22, 2022

Can you approve running on python-poetry/poetry-core#263?

Unfortunately I don't have access yet 😓 , I've just started helping here. I'll see if I can find someone who does

cc @finswimmer?

Any preference on how to navigate this?

Thanks for checking, but it's fine as is! I only have it as a requirement for another project

I believe there's one more version-gated block we can remove:

if sys.version_info < (3, 7) and not is_wheel:
# Under certain Python3.6 installs vendored pip wheel does not contain
# zip-safe pep517 lib. In this cases we create an isolated ephemeral virtual
# environment.
with ephemeral_environment(
executable=environment.python, with_pip=True, with_setuptools=True
) as env:
return environment.run(
*env.get_pip_command(),
*args,
env={**os.environ, "PYTHONPATH": str(env.purelib)},
)

Copy link
Member

@branchv branchv left a comment

Choose a reason for hiding this comment

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

Thanks @tony, this looks good to me. Let's get one more reviewer to approve/merge

@tony
Copy link
Contributor Author

tony commented Jan 22, 2022

Thanks @tony, this looks good to me. Let's get one more reviewer to approve/merge

Thank you for the review!

@finswimmer finswimmer merged commit a0f3d5a into python-poetry:master Jan 23, 2022
@tony tony deleted the deprecate-python-3.6 branch February 5, 2022 20:22
@finswimmer finswimmer mentioned this pull request Mar 6, 2022
mpeteuil added a commit to mpeteuil/poetry-dotenv-plugin that referenced this pull request Sep 3, 2022
Poetry removed 3.6 support in 1.2.0 (as of this commit
python-poetry/poetry#5055), so we can't support
it here either.
mpeteuil added a commit to mpeteuil/poetry-dotenv-plugin that referenced this pull request Sep 3, 2022
Poetry removed 3.6 support in 1.2.0 (as of this commit
python-poetry/poetry#5055), so we can't support
it here either.
mpeteuil added a commit to mpeteuil/poetry-dotenv-plugin that referenced this pull request Sep 3, 2022
Poetry removed 3.6 support in 1.2.0, so we can't support
it here either. This was because 3.6's end-of-life was on December 23,
2021. For more information see:

python-poetry/poetry#5055
https://devguide.python.org/versions/
https://peps.python.org/pep-0494/

In addition to removing 3.6 support, we're moving this project to require Python >=3.7 and <4
mpeteuil added a commit to mpeteuil/poetry-dotenv-plugin that referenced this pull request Sep 3, 2022
* Set bash as the default shell

This removes the necessity of specifying the shell for every step that
uses the shell.

* Use the install script from Poetry's website

* Include 3.10 in tests

* Remove 3.6 support, require >=3.7,<4

Poetry removed 3.6 support in 1.2.0, so we can't support
it here either. This was because 3.6's end-of-life was on December 23,
2021. For more information see:

python-poetry/poetry#5055
https://devguide.python.org/versions/
https://peps.python.org/pep-0494/

In addition to removing 3.6 support, we're moving this project to require Python >=3.7 and <4

* Use the new plugin add syntax

* Create a release GitHub Actions workflow

This will make it simpler to release versions in the future.

* Bump version to remove pre-release
@github-actions
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 29, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Deprecate Python 3.6

4 participants