Skip to content

Conversation

@KnitCode
Copy link
Contributor

not sure whether you'd want to include these change in main repo, but they include corrections and enhancements we use in our work. Addresses the change to the publicsuffix list to be utf-8 not idna-encoded, and contains some variations that we have found to be useful. updated the documentation in the code and README. but right now, it's a bit of a merge.

includes additional functionality for strict checks, ignoring
wildcards, and finding eTLD only. maps main function of
get_public_suffix() to get_sld() for clarity.

KnitCode and others added 7 commits March 27, 2019 13:02
includes additional functionality for strict checks, ignoring
wildcards, and finding eTLD only. maps main function of
get_public_suffix() to get_sld() for clarity.
make test compatible with python2
@KnitCode
Copy link
Contributor Author

KnitCode commented Aug 8, 2019

upgrades pushed to new pypi module, publicsuffix3

@KnitCode KnitCode closed this Aug 8, 2019
@pombredanne
Copy link
Member

@KnitCode I am sorry I had missed that PR entirely!!!
I would definitely prefer that we keep a single module together if this works for you.
How should we proceed?

@pombredanne pombredanne reopened this Aug 8, 2019
@KnitCode
Copy link
Contributor Author

KnitCode commented Aug 8, 2019

@pombredanne ok.. i should have tried to ping you once more before pushing to pypi. someone asked for the ability to see the tld list, so i added that as an attribute this morning. that's only a few lines of code, so I can add that in and push it here.

i can then delete publicsuffix3 on pypi... i'm assuming that won't be hard. ;)

 access the 'list' of TLDs.
 fixes some documentation typos.
version bump to 20190808.
Copy link
Member

@pombredanne pombredanne left a comment

Choose a reason for hiding this comment

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

Looking all good to me.
See the minor nits I made inline... I can handle these alright too.

@pombredanne
Copy link
Member

Thank you ++!

@pombredanne pombredanne merged commit ac14599 into aboutcode-org:develop Aug 11, 2019
@pombredanne pombredanne mentioned this pull request Aug 11, 2019
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