-
Notifications
You must be signed in to change notification settings - Fork 302
Plugin dev: tighten config section (Part 6) #922
Copy link
Copy link
Open
Labels
Description
From PR #761 review by @adamrtalbot (March 26, approval comment):
The config section is still a bit weak and could be tightened but the rest is pretty solid now.
Part 6 (06_configuration.md) covers making the decorator and observer configurable, then adding a formal ConfigScope. Areas to consider tightening:
- The intentional build failure in section 1.1-1.3 (Adam questioned whether it adds value, then said "maybe it's OK to leave it in for now")
- General verbosity of the configuration explanations
- The ConfigScope/navigate() duality explanation could be more concise
🤖 Generated with Claude Code
Reactions are currently unavailable