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 installation documentation links. #133

Merged
merged 7 commits into from May 28, 2014
Merged

Conversation

markkness
Copy link
Contributor

This PR updates some of the links in (mainly) the installation notes. In particular,

  1. Links to EPD are now replaced with links to Canopy.
  2. Links to svn.enthought.com are replaced with something that exists, usually docs.enthought.com.
    I just removed the references to svn.enthought.com/enthought/wiki/Install as there did not seem to be anything to replace this with. I also removed the link to IntelMacPython25 build notes, as this also has no current replacement and is thought to be obsolete.

Please note that I did not try really update the installation notes, this PR just attempts to fix links that are either broken or deprecated, and leaves it at that.

@cgodshall : This should make the changes you desire.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 4302bdf on update-epd-docs into 2054fd4 on master.


is a trac page where one can also enter bug reports and feature
requests.
The Mayavi web page: http://docs.enthought.com/mayavi/mayavi
Copy link
Member

Choose a reason for hiding this comment

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

This should really be the Mayavi documentation and not the "web page" which should really point to either http://mayavi.sf.net or http://code.enthought.com/projects/mayavi

@prabhuramachandran
Copy link
Member

LGTM apart from the minor comment above. Could you please fix that and I will merge this. Thanks!

@markkness
Copy link
Contributor Author

I have changed the descriptive text to 'documentation' (and left the link as is.)

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 75469f2 on update-epd-docs into 2054fd4 on master.

@prabhuramachandran
Copy link
Member

Thanks @markkness! Merging!

prabhuramachandran added a commit that referenced this pull request May 28, 2014
Update installation documentation links.
@prabhuramachandran prabhuramachandran merged commit e76fa0f into master May 28, 2014
@prabhuramachandran prabhuramachandran deleted the update-epd-docs branch May 28, 2014 15:09
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.

None yet

3 participants