Skip to content

Add older variants to pt - #322

Merged
m-mohr merged 6 commits into
mainfrom
pt-2017-2022
Sep 25, 2026
Merged

m-mohr merged 6 commits into
mainfrom
pt-2017-2022

Conversation

@m-mohr

@m-mohr m-mohr commented Sep 24, 2026 •

Copy link
Copy Markdown
Contributor

Reimplementation of #228 by @haithcoatj with additional improvements and cleanups.

@m-mohr m-mohr mentioned this pull request Sep 24, 2026
15 of 20 tasks
Reimplementation of #228 with additional improvements and cleanups.
@m-mohr
m-mohr marked this pull request as ready for review September 24, 2026 14:14

This comment was marked as resolved.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The converter behavior, dependency update, and data-specific edge cases are coherently implemented and comprehensively tested.

Review effort: Balanced
Findings: None

Resolved since last review (4)

@ivorbosloper ivorbosloper left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Check the code repetition element

Comment thread tests/test_convert.py
ivorbosloper added a commit that referenced this pull request Sep 24, 2026
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@m-mohr
m-mohr merged commit 562d623 into main Sep 25, 2026
7 checks passed
@m-mohr
m-mohr deleted the pt-2017-2022 branch September 25, 2026 10:45
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