Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Update README URLs based on HTTP redirects #3552

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

Merged
merged 2 commits into from
Dec 27, 2015
Merged

Update README URLs based on HTTP redirects #3552

merged 2 commits into from
Dec 27, 2015

Conversation

@ethomson
Copy link
Member

I'm always bummed out by automated tools opening pull requests without any human intervention.

I guess that I see some value in saving the end user a few redirects - if every http://www.cmake.org became https://cmake.org/ then we could save the entire Internet perhaps tens of dollars per month.

But these sorts of automated tools are stylistically empty and mundane. Why not just https://cmake.org, which was in keeping with the original URI?

But more importantly, just because one URI redirects to another URI - that doesn't mean that the destination is useful at all. For example, https://wiki.gnome.org/Libgit2-glib is a not found page, which is certainly not any more useful than before.

If an actual human had done any of this, they would have obviously given up a precious 2 seconds of thinking and decided that obviously that was idiotic, and that instead the new URI should be https://wiki.gnome.org/Projects/Libgit2-glib.

So hopefully, there's actually somebody listening to the pull requests that you're opening and responding to them, and this isn't just a spammy robot service. If you fix up the URIs to actually point to the right places, then I'll (begrudgingly) merge this.

@ReadmeCritic
Copy link
Contributor Author

@ethomson updated

@ethomson
Copy link
Member

Thanks for the update. I'm glad to see that there's a real human monitoring the issues that are being opened (and a quite responsive human at that.)

I, for one, don't quite welcome my robotic software writing overloads just yet, but I suppose I should be a bit more lax on human / robot hybrids. ;)

Thanks!

ethomson added a commit that referenced this pull request Dec 27, 2015
Update README URLs based on HTTP redirects
@ethomson ethomson merged commit a066b4c into libgit2:master Dec 27, 2015
@ethomson
Copy link
Member

❤️

@ReadmeCritic
Copy link
Contributor Author

😉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants