docs: rewrite generated docstrings in pipeline.py - #140
Conversation
|
Close — the comments are handled exactly right, and I want to flag that first because it is the Two things before I merge. Three lines from the issue list are still untouched, all
And four of the rewritten lines don't match what the code does. I would rather raise these than
No rush, and none of this is a rewrite — it is seven lines. Push when you have them and I will merge |
|
Thanks for the detailed feedback! I've updated the 3 untouched Returns: lines and corrected the 4 docstrings to accurately reflect function behavior. Pushed the update! |
|
All seven addressed — merging. Apologies for the overnight wait on the corrected push. The one I most wanted right is This is your first contribution here and it took a review round to land — that is completely normal, |
Fixes #136
Rewrote machine-generated docstrings and inline comments in palinode/ingest/pipeline.py to natural English, matching the style established in #131 per issue #130.