Skip to content

[PHP] fix: canonicalize cwd after chdir#721

Closed
brandonpayton wants to merge 1 commit into
Automattic:integration/kd-6nz-php-phpt-harness-only-basefrom
brandonpayton:gascity/kd-6nz/split-pr717-canonicalize-cwd-after-chdir
Closed

[PHP] fix: canonicalize cwd after chdir#721
brandonpayton wants to merge 1 commit into
Automattic:integration/kd-6nz-php-phpt-harness-only-basefrom
brandonpayton:gascity/kd-6nz/split-pr717-canonicalize-cwd-after-chdir

Conversation

@brandonpayton

@brandonpayton brandonpayton commented Jun 20, 2026

Copy link
Copy Markdown
Member

Split from #717 so the PHP PHPT harness PR can be narrowed and later rebased/folded onto separately reviewable platform fixes.

This PR now targets integration/kd-6nz-php-phpt-harness-only-base, the Automattic mirror of #736's harness-only head. It is a PHP-driven platform fix split from #717 and should be reviewed/rebase-merged as a derived fix before folding back into the harness branch.

Original PR #717 commit: 6c459b8446e4219238d1107c730bd3e98cfd2935

Scope: fix: canonicalize cwd after chdir

Validation in this split step:

  • git diff --check origin/main...HEAD

The full PHP PHPT campaign evidence remains attached to #717 / kd-dmm; this PR isolates the platform fix for separate review and rebase-merge preservation.

(cherry picked from commit 02deaca)
(cherry picked from commit 6c459b8)
@brandonpayton brandonpayton changed the base branch from main to integration/kd-6nz-php-phpt-platform-fixes June 20, 2026 17:53
@brandonpayton brandonpayton marked this pull request as ready for review June 20, 2026 17:53
@brandonpayton brandonpayton changed the title fix: canonicalize cwd after chdir [PHP] fix: canonicalize cwd after chdir Jun 20, 2026
@brandonpayton brandonpayton changed the base branch from integration/kd-6nz-php-phpt-platform-fixes to integration/kd-6nz-php-phpt-harness-only-base June 20, 2026 21:07
@brandonpayton

Copy link
Copy Markdown
Member Author

Closing as superseded by origin-headed replacement #741: #741

This PR head was in brandonpayton/kandelo-agentic-dev. The replacement uses the same head SHA (429390c) on an Automattic/kandelo origin branch and has an expanded description explaining why the change is needed and why it belongs in Kandelo.

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