Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

[PhpUnitBridge] skip interactive questions asked by Composer#60616

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
nicolas-grekas merged 1 commit intosymfony:6.4fromxabbuh:pr-60587
Jun 2, 2025

Conversation

@xabbuh
Copy link
Member

QA
Branch?6.4
Bug fix?yes
New feature?no
Deprecations?no
Issues
LicenseMIT

following#60587 so that the installation script is not blocked by Composer asking to install the bridge as a dev dependency:

The package you required is recommended to be placed in require-dev (because it is tagged as "testing") but you did not use --dev.Do you want to re-run the command with --dev? [yes]?

followingsymfony#60587 so that the installation script is not blocked byComposer asking to install the bridge as a dev dependency:```The package you required is recommended to be placed in require-dev (because it is tagged as "testing") but you did not use --dev.Do you want to re-run the command with --dev? [yes]?```
@carsonbotcarsonbot added this to the6.4 milestoneJun 1, 2025
@xabbuhxabbuh changed the titleskip interactive questions asked by Composer[PhpUnitBridge] skip interactive questions asked by ComposerJun 1, 2025
@carsonbotcarsonbot changed the title[PhpUnitBridge] skip interactive questions asked by Composerskip interactive questions asked by ComposerJun 1, 2025
@carsonbotcarsonbot changed the titleskip interactive questions asked by Composer[PhpUnitBridge] skip interactive questions asked by ComposerJun 2, 2025
@nicolas-grekas
Copy link
Member

Why no set --dev instead?

@nicolas-grekas
Copy link
Member

Thank you@xabbuh.

@nicolas-grekasnicolas-grekas merged commit9f3c279 intosymfony:6.4Jun 2, 2025
17 of 18 checks passed
@nicolas-grekas
Copy link
Member

Good, the questions are not displayed anyway :)

@xabbuh
Copy link
MemberAuthor

Why no set --dev instead?

I tried that first but ended of with autoloading issues when running Symfony tests locally and then stopped debugging. 😄

Good, the questions are not displayed anyway :)

In fact they were displayed locally running Symfony's tests. Looks like the behaviour is different in a CI when Composer detects that it runs in a non-interactive environment. 🤷

@nicolas-grekas
Copy link
Member

nicolas-grekas commentedJun 2, 2025•
edited
Loading

maybe we should set COMPOSER_NO_INTERACTION=1 during the call instead?

xabbuh reacted with thumbs up emoji

@xabbuhxabbuh deleted the pr-60587 branchJune 2, 2025 06:58
@xabbuh
Copy link
MemberAuthor

Could be worth to investigate it as a future-proof solution in$passthruOrFail indeed.

@xabbuh
Copy link
MemberAuthor

see#60675

nicolas-grekas added a commit that referenced this pull requestJun 4, 2025
…-interactively (xabbuh)This PR was merged into the 6.4 branch.Discussion----------[PhpUnitBridge] implicitly run all Composer commands non-interactively| Q             | A| ------------- | ---| Branch?       | 6.4| Bug fix?      | no| New feature?  | no| Deprecations? | no| Issues        |Fix#60616 (comment)| License       | MITCommits-------2f1408f implicitly run all Composer commands non-interactively
This was referencedJun 28, 2025
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@OskarStarkOskarStarkOskarStark approved these changes

@derrabusderrabusderrabus approved these changes

@HypeMCHypeMCHypeMC approved these changes

Assignees

No one assigned

Projects

None yet

Milestone

6.4

Development

Successfully merging this pull request may close these issues.

6 participants

@xabbuh@nicolas-grekas@OskarStark@derrabus@HypeMC@carsonbot
[8]ページ先頭
©2009-2025 Movatter.jp