Skip to content

Align Step 2 prompt and screenshot - #5

Open
a-a-ron wants to merge 2 commits into
mainfrom
a-a-ron-add-activity-screenshots
Open

a-a-ron wants to merge 2 commits into
mainfrom
a-a-ron-add-activity-screenshots

Conversation

@a-a-ron

@a-a-ron a-a-ron commented Sep 30, 2026

Copy link
Copy Markdown
Contributor

Summary

  • replace the Step 2 visual with the exact 27-line starter tests/test_bill_splitter.py state and seven-test baseline
  • add a concise, copy-ready Copilot prompt that requests assumptions first and exactly two named tests
  • keep the prompt grounded in the public API, explicit Decimal lists, and no production changes

Validation

  • mechanically matched all 27 screenshot editor lines, including blanks, to tests/test_bill_splitter.py
  • verified both expected result lists against split_bill
  • focused-test grader passes with prompt-compatible tests; resulting suite passes 9 tests
  • image is readable at 1440x900 and the repository-aware rendered URL resolves correctly
  • workflow YAML parsing, Markdown checks, and git diff --check pass

Follow-up to #4, which was merged before this Step 2 feedback was applied.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot AI balanced review requested due to automatic review settings September 30, 2026 22:18

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The prompt matches the public API, grader requirements, expected results, and stated scope.

Review effort: Balanced
Findings: None

What changed in this PR

Aligns the Step 2 learner instructions with the updated screenshot and required focused tests.

Changes:

  • Updates the screenshot description for the seven-test baseline.
  • Adds a copy-ready Copilot prompt requesting assumptions and two exact tests.
File Description
.github/​steps/​2-step.md Updates the Step 2 visual description and test-generation prompt.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
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.

2 participants