docs: reference template method pattern#296
Conversation
Reviewer's guide (collapsed on small PRs)Reviewer's GuideUpdates the stage customization documentation to briefly explain that File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
praise: Thanks so much for the contribution, @lavaFreak 😀. Definitely should've done this ages ago—thanks for taking care of it. todo: Could you please relocate your addition to the end of the stage implementation details page? If a user is reading through the docs sequentially, I'd like it to show up the first time they might realize the template method pattern is in play. |
Just for consistency, and ease of readability.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #296 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 2 2
Lines 177 177
Branches 15 15
=========================================
Hits 177 177 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
Thanks again, @lavaFreak. If there are any other issues you'd like to take a crack at, feel free. As you use the package, do let me know if you run into any problems, usability improvements, etc. |
|
Thanks, Jason. I appreciate the helpful review and the invitation. I'll definitely be looking into some other issues in the repo. |
Summary
Add a short note to the stage customization docs explaining that
StagedScriptfollows the Template Method pattern.Related Issue
Closes #105
What changed
Validation
git diff --check.pytestandruffare not currently installed in this environment.Summary by Sourcery
Document the Template Method pattern usage in stage customization examples.
Documentation: