GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — fix(courses): instruct explicit --attest for reflection checkpoints in all seven tool courses by xxg1413 · Pull Request #113 · flypythoncom/python

Conversation

@xxg1413

@xxg1413 xxg1413 commented Nov 4, 2026

Copy link
Copy Markdown
Member

Problem

The six legacy tool courses (Claude Code, Cursor, DeepSeek Harness, Kimi Code, Codex, ZCode) told learners that the default python verify.py output already contains the L01/L02 claim codes. The verifier requires explicit --attest for self-reported checkpoints, so following the lessons literally never printed the promised codes. L05 said "claim it when you can answer" without giving the command.

Fix

  • L01 (all seven tool courses, EN+ZH): state the default shows [pending] with no code, then instruct python verify.py --attest l01 after answering the checkpoint questions; explain exit code 1 while other checkpoints remain open and that attestations are not persisted between runs.
  • L02 (six legacy courses, EN+ZH): replace "claim code prints when you can answer" with the explicit --attest l02 command (wording matches the Copilot course, which was already correct).
  • L05 (six legacy courses, EN+ZH): give the explicit --attest l05 command.
  • COURSE.md/COURSE_cn.md (all seven): the verify.py teaching contract now describes [passed] vs [attested], non-persistent attestations, exit-code semantics, and the full-attestation completion example; fixed merged Honesty bullet formatting.
  • Bumped content_version and reviewed_on on all touched documents; content-manifest.json regenerated.

Verification

  • make check green: 146 tests, 15 course contracts, 5 examples, 2 paths, all generated artifacts current.
  • Clean-copy behavior tests on all seven courses: default run shows reflections [pending] with no codes; --attest l01 --attest l02 attests exactly those two with codes while l05 stays pending; full flags after implementing the starter exit 0 with all five codes. Every documented command in L01/L02/L05 matches this behavior in both languages.

Verifiers and course code are unchanged; this is a documentation-accuracy fix only.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@xxg1413
fix(courses): instruct explicit --attest for reflection checkpoints i…
e58a4be
…n all seven tool courses
@xxg1413
xxg1413 merged commit 78e8bf7 into master Nov 4, 2026
2 checks passed

Uh oh!

There was an error while loading. Please reload this page.

@xxg1413
xxg1413 deleted the fix/lesson-claim-instructions branch November 4, 2026 05:24
@xxg1413 xxg1413 mentioned this pull request Nov 4, 2026
fix(courses): L01 hints and evidence sections follow the explicit --attest flow #114
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

1 participant

@xxg1413
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.