Skip to content

Conversation

@HassanAbouelela
Copy link
Member

@HassanAbouelela HassanAbouelela commented May 11, 2021

Description

Bumps the python version to 3.9 (.5 in docker), and switches from pipenv to poetry as the package manager. The vote to switch to poetry is here.

Changes

  • Pipfile, Pipfile.lock -> pyproject.toml, poetry.lock
  • Dockerfile Python 3.8 -> 3.9.5
  • Dependency feedparser updated due to a bug in the older version
  • CI Pipenv -> Poetry
  • Documentation Pipenv -> Poetry
  • Add python-dotenv

This is a good time to bump python-discord/sir-lancebot#623 which was stalled by the same dependency this PR was.

Migrates package manager to poetry, and updates python version to 3.9.
Some packages are updated where needed.

Signed-off-by: Hassan Abouelela <hassan@hassanamr.com>
Adds python dotenv to emulate the default .env file loading behavior
from pipenv.

Signed-off-by: Hassan Abouelela <hassan@hassanamr.com>
@HassanAbouelela HassanAbouelela added a: dependencies Related to package dependencies and management a: CI Related to continuous intergration and deployment t: enhancement Changes or improvements to existing features labels May 11, 2021
@HassanAbouelela HassanAbouelela marked this pull request as draft May 11, 2021 18:21
Updates the Dockerfile, pre-commit, CI, and documentation to reflect the
 new dependency manager. Dockerfile is also updated to 3.9.

Signed-off-by: Hassan Abouelela <hassan@hassanamr.com>
Updates Pipfile and pipenv lock file to poetry equivalents in
CODEOWNERS.

Signed-off-by: Hassan Abouelela <hassan@hassanamr.com>
@HassanAbouelela HassanAbouelela marked this pull request as ready for review May 11, 2021 18:33
@Xithrius Xithrius added the p: 3 - low Low Priority label May 12, 2021
Copy link
Member

@jb3 jb3 left a comment

Choose a reason for hiding this comment

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

Lookin' fly :shipit:

@HassanAbouelela HassanAbouelela merged commit 0d425e2 into main May 16, 2021
@HassanAbouelela HassanAbouelela deleted the 39-poetry branch May 16, 2021 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: CI Related to continuous intergration and deployment a: dependencies Related to package dependencies and management p: 3 - low Low Priority t: enhancement Changes or improvements to existing features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants