Skip to content

Try Windows workaround#181

Closed
swissspidy wants to merge 1 commit intomainfrom
try/windows
Closed

Try Windows workaround#181
swissspidy wants to merge 1 commit intomainfrom
try/windows

Conversation

@swissspidy
Copy link
Copy Markdown
Member

No description provided.

@github-actions github-actions bot added scope:distribution Related to distribution scope:testing Related to testing labels Apr 14, 2026
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the wp-cli/wp-cli-tests dependency in composer.json to point to a specific development branch. Feedback indicates that using a branch reference instead of a stable version constraint can lead to environment instability and reproducibility issues, and it should be replaced with a tagged version.

Comment thread composer.json
"wp-cli/entity-command": "^1.3 || ^2",
"wp-cli/extension-command": "^1.2 || ^2",
"wp-cli/wp-cli-tests": "^5"
"wp-cli/wp-cli-tests": "dev-fix/realpath"
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

Referencing a specific branch (dev-fix/realpath) instead of a version constraint makes the development environment unstable and non-reproducible. This should be replaced with a tagged version or a stable constraint before merging to ensure long-term maintainability.

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 14, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@swissspidy swissspidy closed this Apr 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope:distribution Related to distribution scope:testing Related to testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant