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

migrate to poetry #1565

Open
wants to merge 16 commits into
base: master
Choose a base branch
from
Open

Conversation

dulmandakh
Copy link
Contributor

@dulmandakh dulmandakh commented Jun 29, 2024

This PR migrates legacy packaging to Poetry, making dependency management easy.

Depends on:

@erikwrede
Copy link
Member

I'd personally prefer switching to poetry or pdm right away when we make the switch to pyproject.toml What do you think? @dulmandakh

@dulmandakh
Copy link
Contributor Author

I use poetry for my projects. I can help with migration too.

I'll change the PR to use poetry then 😉

@dulmandakh dulmandakh changed the title migrate to pyproject.toml migrate to poetry Jul 5, 2024
@dulmandakh
Copy link
Contributor Author

@erikwrede please review 😉

@@ -46,7 +46,6 @@
from .utils.resolve_only_args import resolve_only_args

__all__ = [
"__version__",
Copy link
Member

Choose a reason for hiding this comment

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

It's important to keep __version__, as certain extension libraries use it do distignuish between different grapehene releases. I'm fine with deprecation, as python 3.8's importlib.metadata.version('graphene') is a viable alternative. I'd suggest using importlib to fill __version__ and deprecating it, recommending direct use of importlib instead. What do you think? 😊

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'll look into it and make changes accordingly

Comment on lines -48 to -53
def get_docs_version(version=None):
version = get_complete_version(version)
if version[3] != "final":
return "dev"
else:
return "%d.%d" % version[:2]
Copy link
Member

Choose a reason for hiding this comment

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

Its also important we keep these in the same format to ensure our docs pipeline doesn't break. Unfortunately, I cannot change much about the way the docs are deployed.

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.

3 participants