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

Update poetry test of oldest supported python version to 3.8 #7691

Merged
merged 1 commit into from
Aug 1, 2023

Conversation

jeffwidman
Copy link
Member

@jeffwidman jeffwidman commented Aug 1, 2023

This test ensures that the installed version of poetry works as expected with the oldest version of python currently supported by Dependabot. It (hopefully) prevents us from installing newer poetry versions that don't have support for older python versions.

In just a couple of weeks we will be deprecating Python 3.6/3.7... So 3.8 will become the oldest version of Python supported by Dependabot.

So update the test accordingly.

To update I ran the following commands:

bin/docker-dev-shell python
pyenv install 3.8.17  # matches the version available to be installed from the `python/Dockerfile`
PYENV_VERSION=3.8.17 pyenv exec pip install --upgrade pip
PYENV_VERSION=3.8.17 pyenv exec pip install poetry
PYENV_VERSION=3.8.17 pyenv exec poetry lock --no-update

Related:

While we can't merge the PR's officially dropping support, we are safe to merge this PR now as it only updates the tests, assuming we're okay with remembering we can't actually bump the minimum poetry version within the next two weeks. I hope that's a reasonable assumption given that I'll be the guy doing the actual bumping. 😁

This test ensures that the installed version of `poetry` works as
expected with the oldest version of python currently supported by
Dependabot. It (hopefully) prevents us from installing newer poetry
versions that don't have support for older python versions.

In just a couple of weeks we will be deprecating Python 3.6/3.7... So
3.8 will become the oldest version of Python supported by Dependabot.

So update the test accordingly.

To update I ran the following commands:
```shell
bin/docker-dev-shell python
pyenv install 3.8.17  # matches the version available to be installed from the `python/Dockerfile`
PYENV_VERSION=3.8.17 pyenv exec pip install --upgrade pip
PYENV_VERSION=3.8.17 pyenv exec pip install poetry
PYENV_VERSION=3.8.17 pyenv exec poetry lock --no-update
```
@jeffwidman jeffwidman marked this pull request as ready for review August 1, 2023 22:15
@jeffwidman jeffwidman requested a review from a team as a code owner August 1, 2023 22:15
@jeffwidman jeffwidman merged commit 4f0c2fa into main Aug 1, 2023
90 checks passed
@jeffwidman jeffwidman deleted the update-poetry-minimum-version-test-to-3.8 branch August 1, 2023 22:30
@jeffwidman jeffwidman added L: elixir:hex Elixir packages via hex Ecosystems Used by the maintainer team for internal-facing project tracking and removed L: python labels Aug 3, 2023
brettfo pushed a commit to brettfo/dependabot-core that referenced this pull request Oct 11, 2023
…dabot#7691)

This test ensures that the installed version of `poetry` works as
expected with the oldest version of python currently supported by
Dependabot. It (hopefully) prevents us from installing newer poetry
versions that don't have support for older python versions.

In just a couple of weeks we will be deprecating Python 3.6/3.7... So
3.8 will become the oldest version of Python supported by Dependabot.

So update the test accordingly.

To update I ran the following commands:
```shell
bin/docker-dev-shell python
pyenv install 3.8.17  # matches the version available to be installed from the `python/Dockerfile`
PYENV_VERSION=3.8.17 pyenv exec pip install --upgrade pip
PYENV_VERSION=3.8.17 pyenv exec pip install poetry
PYENV_VERSION=3.8.17 pyenv exec poetry lock --no-update
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ecosystems Used by the maintainer team for internal-facing project tracking L: elixir:hex Elixir packages via hex
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants