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

Change public suffix list URL #77

Merged

Conversation

mauricioabreu
Copy link
Contributor

The first one is a redirect. This redirect may not live forever so this
change points to the official URL.

The first one is a redirect. This redirect may not live forever so this
change points to the official URL.
@floer32
Copy link
Collaborator

floer32 commented Nov 11, 2015

👍

BTW have you see this bot going around changing redirects in people's READMEs. Kinda fun 😄

@mauricioabreu
Copy link
Contributor Author

Wow, this bot looks amazing.

john-kurkowski added a commit that referenced this pull request Nov 11, 2015
@john-kurkowski john-kurkowski merged commit bd9767b into john-kurkowski:master Nov 11, 2015
@john-kurkowski
Copy link
Owner

Thanks!

This drew my attention to the mirror. There was a big delta! mozilla/gecko-dev's copy of the PSL is stale. Fixed in 0331113.

@mauricioabreu
Copy link
Contributor Author

You are welcome!

@mauricioabreu mauricioabreu deleted the change-suffix-list-url branch November 11, 2015 17:16
@floer32
Copy link
Collaborator

floer32 commented Nov 11, 2015

Great catches, @mauricioabreu , @john-kurkowski

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