Skip to content

Notes on GPL dependencies#129

Closed
C-nit wants to merge 19 commits intoun33k:stagingfrom
C-nit:master
Closed

Notes on GPL dependencies#129
C-nit wants to merge 19 commits intoun33k:stagingfrom
C-nit:master

Conversation

@C-nit
Copy link
Copy Markdown
Contributor

@C-nit C-nit commented Feb 23, 2023

Since there are continued inquiries on the topic of licensing,
an addition to the README.md seems worthwhile to me.

Hopefully this makes the conclusions reached in various previous issues more accessible for the next person.

un33k and others added 19 commits April 27, 2022 11:59
This allows you to use `from slugify import __version__ as slugifyVersion`, while `from slugify.__version__ import __version__ as slugifyVersion` continues to work.
* Add support for Python 3.11

* Drop support for EOL Python 3.6
AS is `text_unidecode` is install_requires it is always present in the python environment, 
it makes sense to try import optinal dependency `unidecode` first, and only then fallback 
to `text_unidecode`.
@C-nit C-nit changed the base branch from master to staging February 23, 2023 15:21
@C-nit C-nit marked this pull request as ready for review February 23, 2023 15:21
@C-nit
Copy link
Copy Markdown
Contributor Author

C-nit commented Feb 23, 2023

@un33k after closing #128 I felt you are irritated by the many issues opened on this topic. I saw those only after the fact.
If the note I propose in the README had been there, I would not have felt the need to open an issue.

@un33k
Copy link
Copy Markdown
Owner

un33k commented Feb 23, 2023

@C-nit Thank you very much for the PR. However, the diffs are not lining up properly.
Could you kindly, cancel this and redo the PR against staging again? Thx

@C-nit C-nit mentioned this pull request Feb 24, 2023
@un33k
Copy link
Copy Markdown
Owner

un33k commented Feb 24, 2023

closed in favor of #130

@un33k un33k closed this Feb 24, 2023
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.

5 participants