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

Update broken/redirecting/unencrypted links #3456

Merged
merged 14 commits into from
Apr 29, 2023

Conversation

pabs3
Copy link
Contributor

@pabs3 pabs3 commented Mar 13, 2023

Many links in gensim are either broken, redirect or are unencrypted.

I made a first pass at updating links in the gensim codebase.

There are many more, I'll submit further patches when I have time.

This is the list of changes in this patchset:

  • Update links to the GNU LGPL v2.1 license
  • Fix links that point to the website about page
  • Update link to gensim install instructions
  • Add trailing slash to links to the website front page
  • Update links to the website to use https
  • Update PyPI module links to pypi.org and https
  • Update PyPI classifiers link to pypi.org, https and HTML
  • Update links to RaRe Technologies website to https
  • Link to the SciPy install page instead of download page
  • Update Wikipedia URLs to https
  • Update SourceForge URLs to https
  • Update GitHub links to https
  • Update links to Google Groups

@pabs3
Copy link
Contributor Author

pabs3 commented Mar 13, 2023

PS: I didn't update any URLs in the images, embedded code copies or generated files. Probably latter two should really be removed from the git repo and instead installed using npm, or built from source during the build.

@mpenkov mpenkov added the documentation Current issue related to documentation label Mar 13, 2023
pabs3 added 13 commits March 14, 2023 07:41
The license was superseded by the LGPL v3 and
consequently the URL of the LGPL v2.1 has changed.

Also update existing correct links to https.

Fixes: commit beb04ea
This page was removed from the website for Gensim 4.

Link instead to the right subsections of the intro page.

Also update the links to https.
The install page now redirects to a section of the index page.

Also update the link to https.

Fixes: commit e17bcf2
Avoids a redirect to URL including the slash.

Also update URLs to https.
This avoids a redirect and protects the content on the network.
Avoids redirects and protects the network data.
Avoids a redirect and protects the content.
The download page no longer exists.

Also update the URL to https.
Support users without JavaScript, avoid redirects, use https.
@piskvorky piskvorky merged commit 525f67a into piskvorky:develop Apr 29, 2023
@piskvorky
Copy link
Owner

Many thanks @pabs3 !

@pabs3 pabs3 deleted the update-links branch April 29, 2023 23:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Current issue related to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants