fix(docs): update mattpocock/skills/grill-me URL#91
Conversation
Matt reorganized the skills repo into category subdirectories; grill-me now lives at skills/productivity/grill-me. Three references (CHANGELOG.md, plugins/teach-me/README.md, plugins/teach-me/commands/quiz.md) all 404'd, which has been failing the lychee link-check on every PR for the last few days. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
Warning Rate limit exceeded
To keep reviews running without waiting, you can enable usage-based add-on for your organization. This allows additional reviews beyond the hourly cap. Account admins can enable it under billing. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughUpdates reference URLs across three documentation files for the teach-me plugin, redirecting the GitHub inspiration source link from a generic path to a more specific skills/productivity directory path. All changes are documentation-only with no functional modifications. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Review rate limit: 0/3 reviews remaining, refill in 3 seconds. Comment |
Old URL 301-redirects to /en. lychee's action fetches the original and gets a bot-detection 500 from the FDPIC site, failing link-check on every PR. Canonical URL renders fine for lychee and humans alike. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
Updates broken documentation links to the reorganized mattpocock/skills repository so CI link-checking (lychee) no longer fails.
Changes:
- Updated the
grill-meGitHub URL in/teach-me:quizcommand docs. - Updated the same URL reference in the
teach-meplugin README. - Updated the same URL reference in the project changelog entry for
teach-mev0.1.0.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| plugins/teach-me/commands/quiz.md | Fixes the grill-me inspiration link to the new skills/productivity/grill-me path. |
| plugins/teach-me/README.md | Fixes the grill-me inspiration link in the command list to the new path. |
| CHANGELOG.md | Fixes the grill-me inspiration link in the Unreleased teach-me entry to the new path. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| # /teach-me:quiz | ||
|
|
||
| Drill yourself on a framework with application-level questions. Ask, evaluate, ask again. Inspired by [`mattpocock/skills/grill-me`](https://github.com/mattpocock/skills/tree/main/grill-me). | ||
| Drill yourself on a framework with application-level questions. Ask, evaluate, ask again. Inspired by [`mattpocock/skills/grill-me`](https://github.com/mattpocock/skills/tree/main/skills/productivity/grill-me). |
There was a problem hiding this comment.
The link target has been updated to the new path, but the visible link text still reads mattpocock/skills/grill-me, which looks like the old location. Consider updating the link text (or making it a more generic label) so it matches the new URL and avoids confusion when someone reads the docs.
| Drill yourself on a framework with application-level questions. Ask, evaluate, ask again. Inspired by [`mattpocock/skills/grill-me`](https://github.com/mattpocock/skills/tree/main/skills/productivity/grill-me). | |
| Drill yourself on a framework with application-level questions. Ask, evaluate, ask again. Inspired by [Matt Pocock's grill-me skill](https://github.com/mattpocock/skills/tree/main/skills/productivity/grill-me). |
| - `/teach-me:control` — pulls a control from the [SCF crosswalk](https://grcengclub.github.io/scf-api/) and explains it once, with the mapped framework controls listed below. Useful when "what does CHG-02 mean and where does it show up?" is the actual question. | ||
| - `/teach-me:role` — what a `grc-engineer`, `grc-auditor`, `grc-internal`, or `grc-tprm` practitioner does day-to-day, what they touch in this toolkit, and where to start. | ||
| - `/teach-me:quiz` — Socratic drill (inspired by [`mattpocock/skills/grill-me`](https://github.com/mattpocock/skills/tree/main/grill-me)). Asks you a control-application question, evaluates your answer, asks another. Tracks which control families you've covered. Stops when you say so. | ||
| - `/teach-me:quiz` — Socratic drill (inspired by [`mattpocock/skills/grill-me`](https://github.com/mattpocock/skills/tree/main/skills/productivity/grill-me)). Asks you a control-application question, evaluates your answer, asks another. Tracks which control families you've covered. Stops when you say so. |
There was a problem hiding this comment.
The URL is updated, but the displayed link text still suggests the old repo path (mattpocock/skills/grill-me). Updating the link text to reflect the new path will make the reference clearer for readers.
| - `/teach-me:quiz` — Socratic drill (inspired by [`mattpocock/skills/grill-me`](https://github.com/mattpocock/skills/tree/main/skills/productivity/grill-me)). Asks you a control-application question, evaluates your answer, asks another. Tracks which control families you've covered. Stops when you say so. | |
| - `/teach-me:quiz` — Socratic drill (inspired by [`mattpocock/skills/productivity/grill-me`](https://github.com/mattpocock/skills/tree/main/skills/productivity/grill-me)). Asks you a control-application question, evaluates your answer, asks another. Tracks which control families you've covered. Stops when you say so. |
| ### Added | ||
|
|
||
| - **`teach-me` plugin (v0.1.0).** Socratic primer + drill plugin for new-to-GRC practitioners and career transitioners — the on-ramp the toolkit was missing for people who don't already know the framework. Four commands: `/teach-me:framework <name>` produces a paraphrased primer (purpose, scope, artifacts, cadence, regulator, control families), `/teach-me:control <id>` explains one control across every framework it maps to via the SCF crosswalk, `/teach-me:role <persona>` outputs a first-90-days onboarding for the four GRC personas, and `/teach-me:quiz <framework>` runs a Socratic drill loop (inspired by [`mattpocock/skills/grill-me`](https://github.com/mattpocock/skills/tree/main/grill-me)) that tracks coverage in-session. Reuses `/grc-engineer:map-controls-unified` and `/grc-engineer:frameworks` rather than duplicating the SCF client. No normative standard text is reproduced. Closes [#61](https://github.com/GRCEngClub/claude-grc-engineering/issues/61). | ||
| - **`teach-me` plugin (v0.1.0).** Socratic primer + drill plugin for new-to-GRC practitioners and career transitioners — the on-ramp the toolkit was missing for people who don't already know the framework. Four commands: `/teach-me:framework <name>` produces a paraphrased primer (purpose, scope, artifacts, cadence, regulator, control families), `/teach-me:control <id>` explains one control across every framework it maps to via the SCF crosswalk, `/teach-me:role <persona>` outputs a first-90-days onboarding for the four GRC personas, and `/teach-me:quiz <framework>` runs a Socratic drill loop (inspired by [`mattpocock/skills/grill-me`](https://github.com/mattpocock/skills/tree/main/skills/productivity/grill-me)) that tracks coverage in-session. Reuses `/grc-engineer:map-controls-unified` and `/grc-engineer:frameworks` rather than duplicating the SCF client. No normative standard text is reproduced. Closes [#61](https://github.com/GRCEngClub/claude-grc-engineering/issues/61). |
There was a problem hiding this comment.
This changelog entry updates the URL, but the visible link label still shows mattpocock/skills/grill-me (the pre-reorg path). Consider updating the link text to match the new location for clarity.
| - **`teach-me` plugin (v0.1.0).** Socratic primer + drill plugin for new-to-GRC practitioners and career transitioners — the on-ramp the toolkit was missing for people who don't already know the framework. Four commands: `/teach-me:framework <name>` produces a paraphrased primer (purpose, scope, artifacts, cadence, regulator, control families), `/teach-me:control <id>` explains one control across every framework it maps to via the SCF crosswalk, `/teach-me:role <persona>` outputs a first-90-days onboarding for the four GRC personas, and `/teach-me:quiz <framework>` runs a Socratic drill loop (inspired by [`mattpocock/skills/grill-me`](https://github.com/mattpocock/skills/tree/main/skills/productivity/grill-me)) that tracks coverage in-session. Reuses `/grc-engineer:map-controls-unified` and `/grc-engineer:frameworks` rather than duplicating the SCF client. No normative standard text is reproduced. Closes [#61](https://github.com/GRCEngClub/claude-grc-engineering/issues/61). | |
| - **`teach-me` plugin (v0.1.0).** Socratic primer + drill plugin for new-to-GRC practitioners and career transitioners — the on-ramp the toolkit was missing for people who don't already know the framework. Four commands: `/teach-me:framework <name>` produces a paraphrased primer (purpose, scope, artifacts, cadence, regulator, control families), `/teach-me:control <id>` explains one control across every framework it maps to via the SCF crosswalk, `/teach-me:role <persona>` outputs a first-90-days onboarding for the four GRC personas, and `/teach-me:quiz <framework>` runs a Socratic drill loop (inspired by [`mattpocock/skills/tree/main/skills/productivity/grill-me`](https://github.com/mattpocock/skills/tree/main/skills/productivity/grill-me)) that tracks coverage in-session. Reuses `/grc-engineer:map-controls-unified` and `/grc-engineer:frameworks` rather than duplicating the SCF client. No normative standard text is reproduced. Closes [#61](https://github.com/GRCEngClub/claude-grc-engineering/issues/61). |
Greptile SummaryThis PR fixes three broken Confidence Score: 5/5Safe to merge — documentation-only link fixes with no logic changes. All changes are hyperlink URL updates in Markdown files. The new grill-me URL is confirmed reachable (200) and the FDPIC URL is a standard authority domain redirect. No code paths, data, or logic are touched. No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A["Lychee link-check CI"] -->|"fails on broken URL"| B["Old: /tree/main/grill-me"]
B -->|"404 / redirect"| C["❌ CI failure on PRs 87, 89, 90"]
A -->|"passes on fixed URL"| D["New: /tree/main/skills/productivity/grill-me"]
D -->|"200 OK"| E["✅ CI passes"]
F["FDPIC URL: /edoeb/en/home.html"] -->|"updated"| G["FDPIC URL: /en"]
subgraph "Files updated"
H["CHANGELOG.md"]
I["plugins/teach-me/README.md"]
J["plugins/teach-me/commands/quiz.md"]
K["plugins/frameworks/ch-fadp/README.md"]
L["plugins/frameworks/ch-fadp/skills/ch-fadp-expert/SKILL.md"]
end
Reviews (1): Last reviewed commit: "fix(ch-fadp): point at canonical FDPIC U..." | Re-trigger Greptile |
Summary
mattpocock/skills/grill-meto the new pathskills/productivity/grill-me. Matt reorganized the skills repo into category subdirectories.Why
This single broken link has been failing the lychee link-check on every PR for the past few days (#87, #89, #90 all hit it). Verified the new URL resolves 200.
Test plan
curl -sI .../skills/productivity/grill-me→ 200🤖 Generated with Claude Code
Summary by CodeRabbit
teach-meplugin to correct documentation URLs.