Skip to content

fix(workshop): raise FK grade level and activity density in 11d2-review-and-merge.md#1066

Merged
pelikhan merged 2 commits into
mainfrom
copilot/curriculum-eval-11d2-review-and-merge-again
Jul 14, 2026
Merged

fix(workshop): raise FK grade level and activity density in 11d2-review-and-merge.md#1066
pelikhan merged 2 commits into
mainfrom
copilot/curriculum-eval-11d2-review-and-merge-again

Conversation

Copilot AI commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

11d2-review-and-merge.md scored FK grade 6.7 (floor: 8) and activity density of 1.04/100 words — driven by short imperative sentences with no explanatory bridging and a single code block across 676 words.

Changes

  • "Review the Pull Request" — added a gh pr list --state open snippet as a terminal alternative to the browser path, and a bridging paragraph explaining that the lock file is the runtime artifact GitHub Actions executes; without it the trigger fires but nothing runs.
gh pr list --state open
  • "Merge the Pull Request" — added a bridging paragraph before the merge steps: after merging, GitHub Actions reads the lock file on every trigger, so both .md and .lock.yml must land on the default branch together or the workflow fails at runtime.

Both additions replace implicit "click and confirm" instructions with causal reasoning, which raises sentence complexity and connects procedural steps to transferable understanding of how the lock file drives execution.

…w-and-merge.md

- Add bridging paragraphs explaining why both .md and .lock.yml must be
  present and why the lock file is the critical runtime artifact
- Add gh pr list --state open shell snippet to increase activity density
- Add explanation before merge steps connecting the merge action to why
  both files must land on the default branch together
- Clarify phrasing per code review feedback

Closes #1062

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Improve readability and activity density in evaluation step fix(workshop): raise FK grade level and activity density in 11d2-review-and-merge.md Jul 14, 2026
Copilot AI requested a review from pelikhan July 14, 2026 13:36
@pelikhan pelikhan marked this pull request as ready for review July 14, 2026 13:38
@pelikhan pelikhan merged commit b1663fa into main Jul 14, 2026
@pelikhan pelikhan deleted the copilot/curriculum-eval-11d2-review-and-merge-again branch July 14, 2026 13:39
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.

[curriculum-eval] 11d2-review-and-merge.md: readability — FK grade 6.7 and low activity density in an evaluate-level step

2 participants