Skip to content

Week 16 教材 - 0611 移除過多提示(提示詞自己打) - #1076

Merged
AugustChaoTW merged 1 commit into
mainfrom
wk16-0611-trim-hints
Jun 10, 2026
Merged

AugustChaoTW merged 1 commit into
mainfrom
wk16-0611-trim-hints

Conversation

@AugustChaoTW

Copy link
Copy Markdown
Contributor

依授課原則調整:0611 教材不應呈現太多等同於「幫學生想好要問什麼」的提示。

  • lab 教案:刪除 timeit 設計提示、必備測試逐條列舉改為「規格每條都要覆蓋,AI 給的自己驗收」
  • starter 骨架:刪除每個測試方法內的提示註解;規格 docstring、紅綠燈流程、subTest 架構要求保留
  • 環境限制(matplotlib Agg)保留——那是事實資訊,不是解題提示
  • 骨架已重新驗證仍為全紅(6 failures)

⚠️ 教材類 PR,CI 必紅,請 admin merge。

🤖 Generated with Claude Code

- 移除 lab 教案的設計提示與必備測試逐條列舉,改為「規格每條都要覆蓋,自己驗收」
- 移除 starter 測試骨架內每個測試的提示註解,保留規格與流程待辦
- 保留環境限制類資訊(matplotlib Agg)與 subTest 共用測試的架構要求

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings June 10, 2026 15:45
@AugustChaoTW
AugustChaoTW merged commit 4e53ff4 into main Jun 10, 2026
1 of 2 checks passed
@AugustChaoTW
AugustChaoTW deleted the wk16-0611-trim-hints branch June 10, 2026 15:45

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

此 PR 依 6/11 課堂教材的授課原則,移除 starter 與 lab 教案中「等同幫學生想好要問什麼」的過多提示文字,同時保留必要的規格描述與環境限制說明(如 matplotlib.use("Agg")),以維持教材的自我探索與驗收導向。

Changes:

  • 移除 test_timing.pytest_sorts.py 中各測試方法內的解題提示註解,保留紅燈骨架與規格導向訊息。
  • 調整 test_sorts.py 規格 docstring,明確區分「被測程式禁用內建排序」與「測試可用 sorted() 作為驗證標準」。
  • 更新 0611-sort-lab.md,將逐條列舉的必備測試提示改為「規格需完整覆蓋、AI 產出需自行驗收」的描述,並保留 CI/無視窗環境限制的事實資訊。

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
weeks/week-16/in_class/0611-sort-starter/test_timing.py 移除測試方法內的提示註解,維持 Stage 1 測試骨架與待辦流程。
weeks/week-16/in_class/0611-sort-starter/test_sorts.py 簡化過度提示、補強規格文字對 sorted()(驗證用途)之說明,維持 subTest 共用測試架構要求。
weeks/week-16/in_class/0611-sort-lab.md 將「必備測試清單」改為「規格全覆蓋+自行驗收」的教材敘述,保留 Agg 等環境限制資訊。

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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