Repository navigation
fix(autograder): never grade seeded workshop issues as Challenge 2 submissions - #243
Conversation
…bmissions The Challenge 2 issue-filed autograder only guarded on author == actor, so peer simulation issues seeded with a user token (and any future App-token seeding, which also triggers workflows) got flagged with challenge-2-needs-revision labels and a Needs revision comment. Skip issues titled Peer Simulation:, Challenge, or Start Here: - those are workshop artifacts, not student submissions. The mislabels and comments this caused in the 8 cohort rooms have been cleaned up in place. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Learning Room Validation ReportGreat work. Your changes look good. Resources |
|
Hi! I am Gandalf, your workshop agent. I bring a little magic to make this interaction clear, helpful, and fun. Here is my review of your pull request: Report StatusValidation Needs Attention [ACTION REQUIRED] Required Checks
Learning ResourcesBased on your changes, these guides might help: Automated validation by Learning Room Bot. Gandalf generated this review. Last updated: 2026-07-04T17:34:31.350Z |
github-actions
Bot
added
documentation
Peer Review AssignedHi @accesswatch! Your PR has been automatically paired with @taylorarndt for peer review. For @taylorarndt:This is a great opportunity to practice code review skills! Here's what to look for: Content Quality:
Accessibility:
Documentation:
Review Guidelines:
Resources: Pairing by Learning Room Grouping Engine |
Progress UpdateGreat work, @accesswatch! Current Level: Explorer Available Challenges:
Keep going! |
There was a problem hiding this comment.
Pull request overview
This PR updates the Learning Room Challenge 2 “issue filed” autograder workflow to avoid treating seeded workshop artifacts (peer simulations, challenge tracker issues, and onboarding roadmap issues) as student Challenge 2 submissions.
Changes:
- Expands the job-level
if:guard to additionally skip issues whose titles start withPeer Simulation:,Challenge, orStart Here:. - Reflows the conditional into a folded multi-line expression for readability and future maintenance.
The Challenge 2 issue-filed autograder only guarded on author == actor,
so peer simulation issues seeded with a user token (and any future
App-token seeding, which also triggers workflows) got flagged with
challenge-2-needs-revision labels and a Needs revision comment. Skip
issues titled Peer Simulation:, Challenge, or Start Here: - those are
workshop artifacts, not student submissions. The mislabels and comments
this caused in the 8 cohort rooms have been cleaned up in place.
Co-Authored-By: Claude Fable 5 noreply@anthropic.com