Skip to content

Conversation

@jonathansick
Copy link
Member

@jonathansick jonathansick commented Mar 10, 2025

Improve user guidance for configuring Git and GitHub access by recommending the gh auth login command for authentication. The gh command is now part of rubin-env, and provides a much easier-to-use workflow for setting GitHub credentials.

Preview: https://rsp.lsst.io/v/DM-49385/guides/notebooks/configuration/git-configuration.html

Other minor updates:

  • Update references from "LSST" to "Rubin"
  • Enhance the introductory paragraph for clarity and friendliness.
  • Adjust visibility of Git LFS setup instructions to reduce confusion.

Copy link
Contributor

@MelissaGraham MelissaGraham left a comment

Choose a reason for hiding this comment

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

As part of this PR I think we should remove this FAQ? Or just replace the answer with a redirect to here?
https://rsp.lsst.io/guides/notebooks/faq/index.html#how-to-make-git-stop-asking-for-my-password

Requests currently has an issue validating the TLS cert for
https://www.star.bris.ac.uk/~mbt/topcat/

This configuration temporarily disables the verification for the
linkcheck.
Since GitHub's gh command is part of rubin-env, we can now use it as the
recommended way of setting up GitHub credentials. This is much more
user-friendly than the credential store method, which we still document
as a backup for other hosts.
- Update "LSST" to "Rubin"
- Make the paragraph on setting up write access to Rubin's Git LFS to be
  visible only in the internal RSPs to avoid confusion.
The intro got mangled in the merger of dp docs. This new intro still
introduces what Git and GitHub are, but strikes a friendlier and more
constructive tone.
@jonathansick jonathansick merged commit 31c4df4 into main Jun 6, 2025
13 checks passed
@jonathansick jonathansick deleted the tickets/DM-49385 branch June 6, 2025 15:12
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.

3 participants