-
-
Notifications
You must be signed in to change notification settings - Fork 532
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
Move installation details from README to the installation guide #38659
base: develop
Are you sure you want to change the base?
Move installation details from README to the installation guide #38659
Conversation
0463362
to
e55a675
Compare
Documentation preview for this PR (built with commit 90bb8fa; changes) is ready! 🎉 |
f3ea24e
to
ccd9dca
Compare
b568039
to
4d0311d
Compare
4d0311d
to
285a6ad
Compare
Thanks for this initiative to make the README lighter! The Linux dependencies listed in the new README are outdated: the Basically, one should suppress A dependencies list that is known to work for Ubuntu 24.04 is posted at https://sagemanifolds.obspm.fr/install_ubuntu.html |
That is not the aim of this PR. The aim is: README for quick walkthrough, installation guide for full details. I think the quick walkthrough is short enough to put in README. Most projects on github contain a simple installation guide in README. |
It's 6 screens long for me... what advantages do you see of having it in readme as opposed to the main documentation?
If we would have a "simple" installation method, I wouldn't mind having a short 3-line cmd in the readme... |
It would be still 6 screens long in the main documentation.
The advantage is "having it in readme". I hope that a newcomer visiting our github repo have the impression that installing sage is not difficult. Our main installation guide is very long and complicated because it has all details explained. I think just looking at the installation guide makes one discouraged. What is the advantage of moving it to the main documentation? Do you want readme to be one screen long? I don't. |
Just to make this clear, my proposal was to move the info that is now in the readme to a new page in the docs. So that it as short/uncomplicated as in the readme.
Automatic checks of links, consistent page layout with other install-related pages, possibility to use advanced features such as tabs, possibility to use semi-dynamic info (such as the auto-generated linux install commands), ... |
74471bc
to
b8499f0
Compare
As stated in the PR description, the aim of this PR is to simplify the instruction guide in readme. README.md is a file directly accessible from the source tree while the installation guide is built from the source tree. I am not sure if removing completely the installation guide from README.md is desirable. As I don't want to do that here, please create your own PR for that later. |
Instead of having two versions in the main installation guide, I hope that the detailed guide is cleaned up and updated. It is now messy and will still be so after this PR. |
can we bump this |
OK. I will revise this soon. |
67ec7b4
to
b38799e
Compare
To repeat, the main point of this PR is to make the README light, keeping the status quo of the traditional build procedures. Making the README even lighter or changing build procedures departing from the status quo should go into new PRs after this one. Please review. |
3bc5049
to
90bb8fa
Compare
Currently we have two docs for installation: README.md and the Sage Installation Guide. There are severe problems:
This PR aims to make README.md a light-weight easy-to-follow installation guide for newcomers and beginners trying to install Sage to their personal computers.
See the new README:
https://github.com/kwankyu/sage/tree/p/streamline-installation-guide#getting-started
The new README does not try to be comprehensive. That is the role of the Sage Installation Guide. Materials removed from README are incorporated to the Sage Installation Guide. The Sage Installation Guide would provide all the details for users for whom the README.md is not sufficient.
The installation guide in the README was tested with macOS, ubuntu, Windows WSL (ubuntu and fedora).
Along the way, I made extensive edits on the Sage Installation Guide. Most of them are non-technical.
See the new Sage Installation Guide:
https://doc-pr-38659--sagemath.netlify.app/html/en/installation/
In future, there would be little need to update the installation guide in README. Developers could focus only on updating the Sage Installation Guide as Sage evolves. This lessens maintenance burden.
📝 Checklist
⌛ Dependencies