Skip to content
This repository has been archived by the owner on Jan 3, 2018. It is now read-only.

Add note about git log options #758

Closed
wants to merge 2 commits into from

Conversation

rgaiacs
Copy link

@rgaiacs rgaiacs commented Sep 27, 2014

Address #745.

@drj11 and @jiffyclub Could you review this?

> a "better" view of the history of your project.
>
> ~~~
> $ git log --oneline --graph --all --decorate
Copy link
Contributor

Choose a reason for hiding this comment

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

--oneline is probably the most useful option at this point, maybe start with just that?

Copy link
Author

Choose a reason for hiding this comment

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

--oneline is probably the most useful option at this point, maybe start with just that?

I don't want to expand this note too much. I hope that instructors already had
taught git help at this point so students can git help log and read what
--oneline, --graph, ... are for.

If more people believe that the write version of the lesson should have this
incremental mention of options I'm OK in change the PR.

Copy link
Contributor

Choose a reason for hiding this comment

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

Sorry, I meant that I think we ought to mention only --oneline and none of the other options.

@rgaiacs
Copy link
Author

rgaiacs commented Sep 28, 2014

@jiffyclub Could you take a look on f162d28?

@rgaiacs
Copy link
Author

rgaiacs commented Sep 30, 2014

@jiffyclub I'm OK to add a note remember students to check --graph at novice/git/03-conflict.md but at this lesson we don't have a example of git log. Will be a little time consuming for me do this change during this split repo thing so could you (1) merge this PR as it is and after #759 I will find some time to improve novice/git/03-conflict.md or (2) could you close this PR and I will create a new one after #759?

@jiffyclub
Copy link
Contributor

Okay, let's close this for now and revisit after we break apart the repo. We'll have #745 to remind us.

@jiffyclub jiffyclub closed this Oct 1, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants