Skip to content

Update extract.custom parameter descriptions - #1062

Draft
ebhills wants to merge 2 commits into
mainfrom
update-docstrings-for-extractdiy-multimatch
Draft

Update extract.custom parameter descriptions#1062
ebhills wants to merge 2 commits into
mainfrom
update-docstrings-for-extractdiy-multimatch

Conversation

@ebhills

@ebhills ebhills commented Jul 12, 2026

Copy link
Copy Markdown
Collaborator
  • incorporated new multi-match capability into recipe wrangle schema
  • made output required --> can't image a case where extract (list) should be slotted into the input (str)
  • add other missing params to core wrangle doc strings

 - added missing params to core wrangle doc strings
 - incorporated new multi-match capability into recipe wrangle schema
@ebhills ebhills added this to the v1.20 milestone Jul 12, 2026
@ebhills
ebhills requested a review from thomasstvr July 12, 2026 11:51
 - add output to required. Never advise to extract into the input column because generally this converts a str to a list
- sequenced the 3 required params as they be expected
@ebhills ebhills self-assigned this Jul 27, 2026
@ebhills
ebhills marked this pull request as draft July 27, 2026 14:03
@ebhills
ebhills removed the request for review from thomasstvr July 27, 2026 14:04

ebhills commented Jul 27, 2026

Copy link
Copy Markdown
Collaborator Author

Queue triage (2026-07-27)

  • Disposition: Draft — delivery-owner action
  • Delivery owner: @ebhills
  • Next action: Fix the failing checks and confirm the schema/API compatibility impact, then request one primary reviewer.

GitHub is the status record; update this PR rather than the external spreadsheet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant