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

[BUGFIX] Fix composer.json #114

Merged
merged 1 commit into from
May 23, 2024
Merged

[BUGFIX] Fix composer.json #114

merged 1 commit into from
May 23, 2024

Conversation

simonschaufi
Copy link
Collaborator

No description provided.

@simonschaufi simonschaufi enabled auto-merge May 23, 2024 13:39
@sabbelasichon sabbelasichon disabled auto-merge May 23, 2024 13:40
@sabbelasichon
Copy link
Collaborator

@simonschaufi What is the reason for that?

@simonschaufi
Copy link
Collaborator Author

it's duplicated now

@sabbelasichon
Copy link
Collaborator

What do you mean with duplicated?

@simonschaufi
Copy link
Collaborator Author

@simonschaufi simonschaufi enabled auto-merge May 23, 2024 13:43
@simonschaufi simonschaufi merged commit 1e5f739 into main May 23, 2024
43 checks passed
@simonschaufi simonschaufi deleted the fix-composer branch May 23, 2024 13:44
@sabbelasichon
Copy link
Collaborator

@simonschaufi To create the composer.json in the root of the repository please run vendor/bin/monorepo-builder merge

@sabbelasichon
Copy link
Collaborator

Normally it should not be necessary to manually fix this.

@simonschaufi
Copy link
Collaborator Author

simonschaufi commented May 23, 2024

You were a bit too fast with merging #109 as I wanted to extract that as I realized that the root composer.json is not covered by the CI atm. That is a but that got introduced when everything moved to the packages folder

@simonschaufi
Copy link
Collaborator Author

vendor/bin/monorepo-builder merge

this should be documented somewhere

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