Skip to content

Conversation

@pvhynes
Copy link
Contributor

@pvhynes pvhynes commented Jan 13, 2026

No description provided.

@pvhynes pvhynes force-pushed the peter/add-extractor-parameters branch from 1ec7f13 to 7fa30c6 Compare January 13, 2026 17:14
@pvhynes pvhynes requested a review from SpencerHedger January 14, 2026 10:20
Copy link
Collaborator

@SpencerHedger SpencerHedger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works well, but some comments to consider.

@pvhynes pvhynes force-pushed the peter/add-extractor-parameters branch from 8a9d7d2 to 9591421 Compare January 15, 2026 11:05
@pvhynes pvhynes requested a review from SpencerHedger January 15, 2026 11:06
Copy link
Collaborator

@SpencerHedger SpencerHedger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just one comment.

@SpencerHedger SpencerHedger self-requested a review January 15, 2026 15:45
Copy link
Collaborator

@SpencerHedger SpencerHedger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

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.

3 participants