Skip to content

Run composer install before npm install for symfony-ux - #494

Closed
BenoitNorrin wants to merge 2 commits into
Scalingo:masterfrom
Dtalents:master
Closed

BenoitNorrin wants to merge 2 commits into
Scalingo:masterfrom
Dtalents:master

Conversation

@BenoitNorrin

Copy link
Copy Markdown

When using symfony-ux, the composer install command must be run before npm install so the JavaScript dependencies can be retrieved in the vendor/ directory.

@Frzk

Frzk commented Nov 12, 2026

Copy link
Copy Markdown
Contributor

Thanks for this PR, and all my apologies for the delay.

  • We've preferred to keep the current behavior and decided to introduce a new environment variable named PHP_BUILDPACK_RUN_COMPOSER_FIRST that will allow you to run Composer before npm kicks in.
    The environment variable value must be set to true.

  • Regarding the support for pdo_sqlsrv, I'd be enclined to accept it if you'd put it in another PR, with an up-to-date version (5.13.3).

Thanks again for your understanding <3

Superseded by #635

@Frzk Frzk closed this Nov 12, 2026
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