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

[git] document configuration of Magit Forge and Git identity #15388

Conversation

practicalli-johnny
Copy link
Contributor

Add documentatation to the Spacemacs Git layer describing how to configure basic git identity and more importantly how to configure Magit Forge to access GitHub / GitLab using a machine configuration and an encrypted file to hold the personal access token.

@real-or-random
Copy link
Contributor

This looks good.

I think your website is also good but it may also be good to link to the official docs, which is the more canonical source and which will probably always up to: date https://magit.vc/manual/forge/Getting-Started.html (We should probably mention that loading the package is of course not required because it's loaded by the layer.)

Copy link
Contributor

@arifer612 arifer612 left a comment

Choose a reason for hiding this comment

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

Thanks for the good work! There are just a few more changes needed to be made.

layers/+source-control/git/README.org Outdated Show resolved Hide resolved
layers/+source-control/git/README.org Outdated Show resolved Hide resolved
@practicalli-johnny practicalli-johnny marked this pull request as draft March 2, 2022 14:01
@practicalli-johnny
Copy link
Contributor Author

I've converted to draft and will think about this some more over the next few days. Thanks.

@real-or-random
Copy link
Contributor

I've converted to draft and will think about this some more over the next few days. Thanks.

Ok. Please don't feel discouraged by the comments, these are really minor points and it's a great thing to add forge docs!

@practicalli-johnny practicalli-johnny marked this pull request as ready for review March 2, 2022 15:15
Copy link
Contributor

@real-or-random real-or-random left a comment

Choose a reason for hiding this comment

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

Looks good to me except a few typos

layers/+source-control/git/README.org Outdated Show resolved Hide resolved
layers/+source-control/git/README.org Outdated Show resolved Hide resolved
layers/+source-control/git/README.org Outdated Show resolved Hide resolved
Copy link
Contributor

@arifer612 arifer612 left a comment

Choose a reason for hiding this comment

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

There's still a typo in one of the links that, but other than that I think the issues that @real-or-random and I pointed out were handled!

layers/+source-control/git/README.org Outdated Show resolved Hide resolved
@practicalli-johnny practicalli-johnny force-pushed the git-magit-forge-documentation branch 2 times, most recently from 2db2fa6 to f86909b Compare March 7, 2022 16:29
Copy link
Contributor

@arifer612 arifer612 left a comment

Choose a reason for hiding this comment

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

Nice! All looks great!

layers/+source-control/git/README.org Outdated Show resolved Hide resolved
layers/+source-control/git/README.org Outdated Show resolved Hide resolved
layers/+source-control/git/README.org Outdated Show resolved Hide resolved
@lebensterben lebensterben merged commit d751c09 into syl20bnr:develop Mar 11, 2022
@lebensterben
Copy link
Collaborator

Merged. Thanks for everyone.

@practicalli-johnny practicalli-johnny deleted the git-magit-forge-documentation branch March 21, 2022 15:04
zv added a commit to zv/spacemacs that referenced this pull request Apr 17, 2022
* checkversion/develop: (143 commits)
  fix typo (syl20bnr#15470)
  Remove disabled code and FIXME: resolved. (syl20bnr#15464)
  disable evil-surround in magit-status buffers (syl20bnr#15462)
  core-jump: fix incorrect lookup of :async (syl20bnr#15449)
  Replace alist helm sources with proper helm sources (syl20bnr#15364)
  Fix syl20bnr#15306. Reflect eaf layer keybinding logic to upstream changes.
  [lsp] Change default source of bindings back to spacemacs
  [core] Make startup not fail when home buffer is customised
  [spacemacs-editing] initialize `undo-tree-history-directory-alist`
  [neotree] Add zz, zt, zb bindings
  notmuch: Fix company based address completion (syl20bnr#15392)
  [bot] "documentation_updates" Mon Mar 28 11:29:34 UTC 2022 (syl20bnr#15433)
  [github] clarify Git configuration and authorisation documentation
  [org] Make org buffers work with helm-imenu-in-all-buffers
  [nixos] Fix variable name in README (syl20bnr#15413)
  [helm] fix typo in helm-ls-git (syl20bnr#15411)
  [completion] Fix readme markup. (syl20bnr#15410)
  [bot] "documentation_updates" Fri Mar 11 06:55:25 UTC 2022 (syl20bnr#15402)
  [git] document configuration of Magit Forge and Git identity (syl20bnr#15388)
  README: Added a Discord Channel
  ...
eraad pushed a commit to datil/spacemacs that referenced this pull request Jun 10, 2022
…r#15388)

Add documentatation to the Spacemacs Git layer describing how to configure basic git identity and more importantly how to configure Magit Forge to access GitHub / GitLab using a machine configuration and an encrypted file to hold the personal access token.
2ynn pushed a commit to 2ynn/spacemacs that referenced this pull request Jun 22, 2022
…r#15388)

Add documentatation to the Spacemacs Git layer describing how to configure basic git identity and more importantly how to configure Magit Forge to access GitHub / GitLab using a machine configuration and an encrypted file to hold the personal access token.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants