Skip to content

Tjmadonna/clt documentation#82

Merged
yuanzhou merged 3 commits intomainfrom
tjmadonna/clt-documentation
Mar 17, 2026
Merged

Tjmadonna/clt documentation#82
yuanzhou merged 3 commits intomainfrom
tjmadonna/clt-documentation

Conversation

@tjmadonna
Copy link
Contributor

See CLT cards #13 #14 #15

Changes

  • Added login --no-browser flag for headless environments
  • Added installation instructions for virtual environments and pipx
  • Added documentation on the transfer --from-protected-space (This was from a couple months ago but was never added)

- Added documentation for login --no-browser command flag
- Added documentation for installing in virtual environments and with pipx
@tjmadonna tjmadonna requested review from shirey and yuanzhou March 12, 2026 14:58
@yuanzhou yuanzhou requested a review from DerekFurstPitt March 12, 2026 19:08
Copy link
Contributor

@DerekFurstPitt DerekFurstPitt left a comment

Choose a reason for hiding this comment

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

Everything looks good. I'll point out regarding "log in" vs "login" that, "log in" is a verbal phrase, and login is a noun. I think its appropriate to say "the user can log in to..." and "this takes the user to the login page". Similar to "set up" vs "setup". But that's just grammatical semantics and largely a matter of personal preference. I'm good with it either way

@yuanzhou yuanzhou merged commit 6b775fa into main Mar 17, 2026
@yuanzhou yuanzhou deleted the tjmadonna/clt-documentation branch March 17, 2026 13:52
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