full-day workshop: Add the missing commit step before creating the feature pull request
Nobody has claimed this yet.
Assessment
- Difficulty
- 1/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 88/100
- Issue type
- Documentation
- Clarity
- Clearly specified
- Activity status
- Active
- Tech stack
- git, github
- Domain
- documentation
Research direction
Open content/full-day/7-github-flow.md and follow the workshop sequence around add-filter, Generate Commit Message, git push, and gh pr create. Add the missing commit and verification instructions so a fresh run confirms the feature commit is ahead of main, pushes it, and successfully creates the pull request.
Written by the indexing model from the issue text.
Description
Problem
In content/full-day/7-github-flow.md, learners create add-filter, run git add ., select Generate Commit Message, press Enter, and then run git push -u origin add-filter. No git commit command or Source Control commit action is provided.
Generating a commit message does not itself create a commit. The branch therefore has no commits ahead of main, and gh pr create can fail because there are no commits between the branches. This blocks the pull-request portion of the workshop.
Suggested change
Add an explicit commit action after generating the message, and verify the branch is ahead of main before pushing and opening the PR.
Acceptance criteria
- Learners are told exactly where the generated message appears.
- An explicit CLI or Source Control commit step is included.
- The instructions verify the feature commit exists on
add-filter. -
git pushpublishes the feature commit. -
gh pr createsucceeds from a fresh workshop run.
- Dominant language
- Python
- Stars
- 80
- Forks
- 161
- Avg merge
- 31m
- Merged PRs (30d)
- 1
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from github-samples/pets-workshop
-
priority: P0
Difficulty 2/5 1-2 days Newbie friendliness 78/100
github-samples/pets-workshop#268 · 1 comment ·
-
priority: P1
Difficulty 1/5 1-3 hours Newbie friendliness 92/100
github-samples/pets-workshop#261 · 1 comment ·
-
priority: P1
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
github-samples/pets-workshop#257 · 1 comment ·
-
priority: deferred
Difficulty 4/5 3-5 days Newbie friendliness 45/100
github-samples/pets-workshop#278 · 1 comment ·
-
priority: deferred
Difficulty 4/5 3-5 days Newbie friendliness 68/100
github-samples/pets-workshop#277 · 1 comment ·
All issues in github-samples/pets-workshop
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
-
enhancement
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100