Skip to content

Fix: Windows quoting (F10) and table inputs (F2 F3 F4 F7), merged and conflict-resolved together - #47

Merged
guijacquemet merged 10 commits into
mainfrom
claude/fix-remaining
Oct 8, 2026
Merged

guijacquemet merged 10 commits into
mainfrom
claude/fix-remaining

Conversation

@guijacquemet

Copy link
Copy Markdown
Contributor

Integration branch: PR #44 (Windows quoting, F10) and PR #45 (table inputs, F2 F3 F4 F7) merged on top of the current main (lifecycle #42, shapes #43 and notebook form #46 already merged). Every fix PR removes its findings from the same lists in tests/roundtrip_known_failures.py and from docs/REGRESSION_LEDGER.md, so merging them one at a time meant a conflict and a 10-minute CI cycle after each. The two PRs' own content and descriptions are unchanged; see #44 and #45. I will merge this with a merge commit so each fix keeps its own commits, then close #44 and #45 as included. The error-codes PR (#41) is rebased on this branch next.

Checks before opening: black, ruff, mypy clean; quoting vectors, table input, known-failures and ledger tests pass locally. CI on the integration head is the real check (the tables fix runs under pandas 2.3.3 on the Python 3.10 jobs).

🤖 Generated with Claude Code

https://claude.ai/code/session_01Y2SN8SuEakTdiWy3F2WUqN


Generated by Claude Code

claude added 10 commits October 8, 2026 12:41
…dable errors (F2, F3, F4, F7)

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Y2SN8SuEakTdiWy3F2WUqN
…ectors (F10)

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Y2SN8SuEakTdiWy3F2WUqN
… float)

No existing test edited in this commit.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Y2SN8SuEakTdiWy3F2WUqN
…ns with missing cells are handled like pandas 3

No existing test edited.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Y2SN8SuEakTdiWy3F2WUqN
@guijacquemet
guijacquemet merged commit 68f4074 into main Oct 8, 2026
14 checks passed
guijacquemet added a commit that referenced this pull request Oct 14, 2026
…y) (#134)

All three hosts honour the app_hosts_v1 rule (Napari #61, QuPath #35, Fiji #47 merged); template #33 is not merged. Sweeps README, HOST_FEATURES, REPOSITORIES, PROTOCOL, HUMAN_TEST_PROTOCOL, TESTING, the release sandbox README and two case files; OPERATIONS: Napari row (no main.zip since #60, placeholder <NAPARI_TAG>) and planned release numbers; TESTING: Known CI flake note.


Claude-Session: https://claude.ai/code/session_01Y2SN8SuEakTdiWy3F2WUqN

Co-authored-by: Claude <noreply@anthropic.com>
@guijacquemet
guijacquemet deleted the claude/fix-remaining branch October 14, 2026 16:08
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