[path] Add runtime-aware tooling binary fallback#293
Conversation
|
🚀 Preview is available for this pull request. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: bce7685277
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex review |
|
Codex Review: Didn't find any major issues. Can't wait for the next one! ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Related Issue
Closes #292
Motivation / Context
dev-toolsruns still assumed the consumer repository provided every tooling binary undervendor/binchangelogdo not installphp-cs-fixer,rector,ecs,jack, orcomposer-dependency-analyserlocally, so global runs could not start those processesChanges
DevToolsPathResolver, preferring the working project'svendor/bin/<tool>when present and falling back to the active DevTools runtime binary path otherwisecode-style,phpdoc,refactor, anddependenciesto use the shared resolver instead of hardcodedvendor/bin/...stringsVerification
composer dev-toolscomposer dev-tools tests -- --filter='(DevToolsPathResolverTest|CodeStyleCommandTest|RefactorCommandTest|PhpDocCommandTest|DependenciesCommandTest)',composer dev-tools phpdoc,composer dev-tools code-styleDocumentation / Generated Output
docs/updatedChangelog
CHANGELOG.mdentryReviewer Notes