Skip to content
This repository has been archived by the owner on Apr 23, 2024. It is now read-only.

Add minimum requirements for package installation #103

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gbeckenkamp
Copy link

No description provided.

… a clean environment can install the package
Copy link

@cdeln cdeln left a comment

Choose a reason for hiding this comment

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

Tried it out in fresh virtual env with python 3.8.
This solves downstream package consumers using pip with --use-pep517 flag (default when using tools such as poetry).

@miguelvalente
Copy link

Tried it out in fresh virtual env with python 3.8. This solves downstream package consumers using pip with --use-pep517 flag (default when using tools such as poetry).

Hi guys. Having the same issue here. Any updates on the status of the merge?

@gbeckenkamp
Copy link
Author

Sorry guys I was away for a while.
I cannot merge due to: At least 1 approving review is required by reviewers with write access.

@Dominux
Copy link

Dominux commented Feb 18, 2023

Just yet another one greatly introduced package, that's "blazingly fast", but you can't just install it LMAO

vitalymegabyte added a commit to vitalymegabyte/YouTokenToMe that referenced this pull request Feb 28, 2023
@gburlet
Copy link

gburlet commented May 11, 2023

This PR has already fallen out of date.

Updated PR here: #108

adrianlzt added a commit to assistify-ai/YouTokenToMe that referenced this pull request Jan 8, 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.

6 participants