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/reorder git instructions in Developer's Guide #33648

Closed
mkoeppe opened this issue Apr 6, 2022 · 13 comments
Closed

Update/reorder git instructions in Developer's Guide #33648

mkoeppe opened this issue Apr 6, 2022 · 13 comments

Comments

@mkoeppe
Copy link
Contributor

mkoeppe commented Apr 6, 2022

(from #33088)

In the developer guide, move "git the hard way" earlier and present "git-trac" only as an additional convenience.
Also note that the first configuration using sagetrac-mirror is not recommended for users who use VS Code (#33589 comment:13)

Depends on #33088

CC: @tobiasdiez @sagetrac-tmonteil @seblabbe @dimpase @yuan-zhou

Component: documentation

Author: Matthias Koeppe

Branch/Commit: aa9faaa

Reviewer: Dima Pasechnik

Issue created by migration from https://trac.sagemath.org/ticket/33648

@mkoeppe mkoeppe added this to the sage-9.6 milestone Apr 6, 2022
@mkoeppe

This comment has been minimized.

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Apr 8, 2022

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Apr 8, 2022

Author: Matthias Koeppe, ...

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Apr 8, 2022

Commit: 88b04d4

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Apr 8, 2022

Last 10 new commits:

b46c953README: Document --enable-editable and --enable-download-from-upstream-url
a3ffe84README.md: Instructions for Apple Silicon
d7708dfREADME.md: Restructure macOS prep instructions
2e1d22dsrc/doc/en/installation/source.rst: Remove instructions for Solaris, AIX, HP-UX
9ca39a2README.md: Point to developer guide for git details
f1e1166src/doc/en/installation/source.rst: Point to README.md for step-by-step instructions
3f6c6e0src/doc/en/developer/workspace.rst: Link to Trac #30500
8716d80src/doc/en/installation/source.rst: Update required venv Python extension modules from build/pkgs/python3/spkg-configure.m4
f4c3604Merge #33088
88b04d4src/doc/en/developer/manual_git.rst: Do not recommend mixed sagetrac-mirror / trac operation when using VS Code

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Apr 9, 2022

Branch pushed to git repo; I updated commit sha1. New commits:

fc7aa4fsrc/doc/en/developer/advanced_git.rst: Mention git log --first-parent
aa9faaasrc/doc/en/developer: move "git the hard way" earlier, present "git-trac" only as an additional convenience

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Apr 9, 2022

Changed commit from 88b04d4 to aa9faaa

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Apr 9, 2022

Changed author from Matthias Koeppe, ... to Matthias Koeppe

@dimpase
Copy link
Member

dimpase commented Apr 9, 2022

Reviewer: Dima Pasechnik

@dimpase
Copy link
Member

dimpase commented Apr 9, 2022

comment:6

lgtm

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Apr 9, 2022

comment:8

Thanks!

@mkoeppe

This comment has been minimized.

@vbraun
Copy link
Member

vbraun commented Apr 21, 2022

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

No branches or pull requests

3 participants