Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Exclude composer.lock from exported archive to avoid lockfile induced installation failure. #137

Merged

Conversation

Xerkus
Copy link
Member

@Xerkus Xerkus commented Jan 9, 2024

Q A
Documentation no
Bugfix yes/no
BC Break no
New Feature yes
RFC no
QA no

Description

Composer lockfile is preventing successful installation after composer create-project. This attempts to remove lockfile, however installer is invoked from pre-install hook and it potentially can be too late to remove lockfile.

@Xerkus Xerkus added the Bug Something isn't working label Jan 9, 2024
@Xerkus
Copy link
Member Author

Xerkus commented Jan 9, 2024

Unfortunately it is too late to remove lockfile from pre-install event.

@Xerkus Xerkus marked this pull request as draft January 9, 2024 21:49
Signed-off-by: Aleksei Khudiakov <aleksey@xerkus.pro>
@Xerkus Xerkus force-pushed the feature/remove-lockfile-before-install branch from 3d1a45e to c5a19c0 Compare January 9, 2024 23:31
@Xerkus Xerkus changed the title Add installer step removing composer.lock to avoid lockfile induced installation failure. Exclude composer.lock from exported archive to avoid lockfile induced installation failure. Jan 9, 2024
@Xerkus Xerkus marked this pull request as ready for review January 9, 2024 23:32
@Xerkus
Copy link
Member Author

Xerkus commented Jan 9, 2024

Best option for now is export-ignore lockfile for the default use of composer create-project.

@Xerkus Xerkus added this to the 3.15.0 milestone Jan 9, 2024
@Xerkus Xerkus merged commit 2cecef9 into mezzio:3.15.x Jan 9, 2024
5 checks passed
@Xerkus Xerkus deleted the feature/remove-lockfile-before-install branch January 9, 2024 23:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

No dependencies in newly created project Composer create project fails with dependency resolution error
1 participant