Skip to content

Conversation

alex-ball
Copy link
Contributor

In episode three, we explain that our first, longer use of git push allows us to use a shorter form later, but we don't call back to that explanation when it happens. There are several ways we could do so, but I think using a challenge would be good as the lesson doesn't have very many of them.

Closes #166.

Closes LibraryCarpentry#166. This lesson has few challenges, so this issue seemed like a good opportunity to add one.
Copy link

github-actions bot commented Aug 4, 2025

Thank you!

Thank you for your pull request 😃

🤖 This automated message can help you check the rendered files in your submission for clarity. If you have any questions, please feel free to open an issue in {sandpaper}.

If you have files that automatically render output (e.g. R Markdown), then you should check for the following:

  • 🎯 correct output
  • 🖼️ correct figures
  • ❓ new warnings
  • ‼️ new errors

Rendered Changes

🔍 Inspect the changes: https://github.com/LibraryCarpentry/lc-git/compare/md-outputs..md-outputs-PR-182

The following changes were observed in the rendered markdown documents:

 03-sharing.md | 33 +++++++++++++++++++++++++++++++++
 md5sum.txt    |  2 +-
 2 files changed, 34 insertions(+), 1 deletion(-)
What does this mean?

If you have source files that require output and figures to be generated (e.g. R Markdown), then it is important to make sure the generated figures and output are reproducible.

This output provides a way for you to inspect the output in a diff-friendly manner so that it's easy to see the changes that occur due to new software versions or randomisation.

⏱️ Updated at 2025-08-14 08:21:03 +0000

github-actions bot pushed a commit that referenced this pull request Aug 4, 2025
Copy link
Contributor

@srerickson srerickson left a comment

Choose a reason for hiding this comment

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

I'm offering slightly more in-depth language for the solution:

Branches can be configured with an "upstream" branch (a branch on a remote
repository), which is used automatically for `push` and `pull` operations. The
`main` branch's upstream wasn't set when we created it. The `-u` (or
`--set-upstream`) flag tells `git push` which remote branch to use; it also sets
the remote branch as the local branch's upstream. Once we've set it, we don't
need to specify the remote branch in future `push` and `pull` operations.

Feel free to incorporate it, or not. Otherwise this, looks good!

@srerickson srerickson assigned alex-ball and unassigned srerickson Aug 12, 2025
Incorporates wording from @srerickson. Thank you!
github-actions bot pushed a commit that referenced this pull request Aug 14, 2025
@alex-ball
Copy link
Contributor Author

Thank you for the more thorough explanation, which I've incorporated.

I think this reinforces the point that we need to find somewhere to explain what a branch is, as touched on in #54.

@alex-ball alex-ball merged commit 16a41d4 into LibraryCarpentry:main Aug 14, 2025
3 checks passed
github-actions bot pushed a commit that referenced this pull request Aug 14, 2025
Auto-generated via `{sandpaper}`
Source  : 16a41d4
Branch  : main
Author  : Alex Ball <alex-ball@users.noreply.github.com>
Time    : 2025-08-14 08:32:08 +0000
Message : Merge pull request #182 from alex-ball/patch-166

Add challenge around "git push"
github-actions bot pushed a commit that referenced this pull request Aug 14, 2025
Auto-generated via `{sandpaper}`
Source  : a27842a
Branch  : md-outputs
Author  : GitHub Actions <actions@github.com>
Time    : 2025-08-14 08:32:50 +0000
Message : markdown source builds

Auto-generated via `{sandpaper}`
Source  : 16a41d4
Branch  : main
Author  : Alex Ball <alex-ball@users.noreply.github.com>
Time    : 2025-08-14 08:32:08 +0000
Message : Merge pull request #182 from alex-ball/patch-166

Add challenge around "git push"
github-actions bot pushed a commit that referenced this pull request Aug 19, 2025
Auto-generated via `{sandpaper}`
Source  : a27842a
Branch  : md-outputs
Author  : GitHub Actions <actions@github.com>
Time    : 2025-08-14 08:32:50 +0000
Message : markdown source builds

Auto-generated via `{sandpaper}`
Source  : 16a41d4
Branch  : main
Author  : Alex Ball <alex-ball@users.noreply.github.com>
Time    : 2025-08-14 08:32:08 +0000
Message : Merge pull request #182 from alex-ball/patch-166

Add challenge around "git push"
github-actions bot pushed a commit that referenced this pull request Aug 26, 2025
Auto-generated via `{sandpaper}`
Source  : a27842a
Branch  : md-outputs
Author  : GitHub Actions <actions@github.com>
Time    : 2025-08-14 08:32:50 +0000
Message : markdown source builds

Auto-generated via `{sandpaper}`
Source  : 16a41d4
Branch  : main
Author  : Alex Ball <alex-ball@users.noreply.github.com>
Time    : 2025-08-14 08:32:08 +0000
Message : Merge pull request #182 from alex-ball/patch-166

Add challenge around "git push"
github-actions bot pushed a commit that referenced this pull request Sep 2, 2025
Auto-generated via `{sandpaper}`
Source  : a27842a
Branch  : md-outputs
Author  : GitHub Actions <actions@github.com>
Time    : 2025-08-14 08:32:50 +0000
Message : markdown source builds

Auto-generated via `{sandpaper}`
Source  : 16a41d4
Branch  : main
Author  : Alex Ball <alex-ball@users.noreply.github.com>
Time    : 2025-08-14 08:32:08 +0000
Message : Merge pull request #182 from alex-ball/patch-166

Add challenge around "git push"
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.

How to use "git push..."
2 participants