Skip to content

Remove the Python divergence entries closed by idfkit#183 - #47

Merged
samuelduchesne merged 1 commit into
mainfrom
remove-python-183-entries
Oct 26, 2026
Merged

samuelduchesne merged 1 commit into
mainfrom
remove-python-183-entries

Conversation

@samuelduchesne

Copy link
Copy Markdown
Contributor

idfkit no longer stores a blank fixed field as "" (idfkit/idfkit#183, fixed in idfkit/idfkit#242). The model now agrees with write_idf and write_epjson, so re-parsing idfkit's own IDF output equals the original. That makes two python / round-trip entries stale:

  • numeric-zero-against-blank
  • positional-trailing-unset-no-extensible

This removes both, along with the positional section header they leave empty, and recounts the register header. The register now holds 10 entries over 5 issues and 7 cases: 5 against python, 5 against typescript. No typescript entry changes.

Verification

  • python -m pytest runners/tests: 229 passed.
  • python runners/run.py --library <idfkit> against this branch, where <idfkit> is a scratch merge of idfkit#236 (IDFObject.update) and idfkit#242: PASS, exit 0. 74 cases, 226 assertions: 218 passed, 5 failed, all 5 of them outstanding exceptions (#186, #187). No stale entries and no unexpected failures. The idfkit unit suite passes on the same merge.
  • Against idfkit#242 alone, this register reports no stale entries. The only unexpected failures are the five update cases, which need IDFObject.update from idfkit#236.

After merge

Cut conformance-2026.20 on the merge commit. idfkit#242 then sets [tool.idfkit.conformance] level to it. Because levels from conformance-2026.17 on include the update cases, #242 adopts the new level once idfkit#236 has landed on idfkit main.

Per CODEOWNERS (FR-067), a change to the register needs review from a maintainer of the other language.


Generated by Claude Code

idfkit no longer stores a blank fixed field as "" (idfkit#183,
idfkit#242), so the model agrees with write_idf and write_epjson and a
re-parse of its own IDF output equals the original. The python
round-trip entries for numeric-zero-against-blank and
positional-trailing-unset-no-extensible are therefore stale and are
removed, together with the positional section they leave empty.

The register now holds 10 entries over 5 issues and 7 cases, 5 against
python and 5 against typescript.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01LEegXpWqwRPaofngRx6L3h
@samuelduchesne
samuelduchesne requested review from a team as code owners October 26, 2026 17:40
@samuelduchesne
samuelduchesne merged commit 7b64f20 into main Oct 26, 2026
1 check passed
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.

2 participants