Skip to content

Added a module to set up git#7

Closed
VickyRampin wants to merge 1 commit intoLibraryCarpentry:gh-pagesfrom
VickyRampin:setup-patch
Closed

Added a module to set up git#7
VickyRampin wants to merge 1 commit intoLibraryCarpentry:gh-pagesfrom
VickyRampin:setup-patch

Conversation

@VickyRampin
Copy link
Copy Markdown

Hi! I added a short lesson on setting up git locally (going through config and briefly explaining global) so learners can do it with us (previously, they did it on their own).

Comment thread _episodes/02-setup.md

One of the main barriers to getting started with git is the language. Although some of the language used in git is
fairly self-explanatory, other terms are not so clear. The best way to get to learn the language - which consists of a
number of verbs such as `add`, `commit` and `push` (precdeded by the word 'git') - is by using it, which is what we will be doing during this
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

typo: "precdeded"

Comment thread _episodes/02-setup.md
$ git config --help
~~~
{: .bash}
{: .callout}
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I think you have to blockquote the whole thing to use the .callout class, like is done here. You also need the titles to be level 2 (## not ###) for some reason.

@katrinleinweber
Copy link
Copy Markdown
Contributor

katrinleinweber commented Dec 20, 2018

Dear @VickySteeves, Because of a transition of the maintainer team here, this PR has not received the attention it deserved, I'm sorry :-( Please have a look at #43 for further discussion about this whole setup topic. That was not a useful idea, sorry! People should help review this PR, of course.

wickr added a commit to wickr/lc-git that referenced this pull request May 22, 2019
Spotted first in review in LibraryCarpentry#7
@wickr wickr mentioned this pull request May 22, 2019
@dcmcand
Copy link
Copy Markdown
Contributor

dcmcand commented Jul 16, 2020

This section has already been added, so I am going to close this PR.

@dcmcand dcmcand closed this Jul 16, 2020
tobyhodges pushed a commit that referenced this pull request Apr 17, 2023
zkamvar pushed a commit that referenced this pull request Apr 21, 2023
Spotted first in review in #7
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.

4 participants