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

Scope cs-fixer to avoid dependency hell #132

Merged
merged 1 commit into from
Mar 3, 2022

Conversation

ChristophWurst
Copy link
Member

@ChristophWurst ChristophWurst commented Mar 1, 2022

The cs-fixer pulls in symfony and friends, which gets autoloaded with
the Nextcloud process and causes incompatibilities with Nextcloud's copy
of Symfony. There is no reason to autoload this by default. We can use
the composer bin plugin instead.

Fixes #121

@ChristophWurst ChristophWurst added the bug Something isn't working label Mar 1, 2022
@ChristophWurst ChristophWurst added this to the 0.3.0 milestone Mar 1, 2022
@ChristophWurst ChristophWurst added this to TO REVIEW (max 4 PRs) in Christoph's Tasks via automation Mar 1, 2022
@ChristophWurst ChristophWurst self-assigned this Mar 1, 2022
nickvergessen
nickvergessen previously approved these changes Mar 1, 2022
Copy link
Member

@nickvergessen nickvergessen left a comment

Choose a reason for hiding this comment

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

Yolo

Christoph's Tasks automation moved this from TO REVIEW (max 4 PRs) to TO INTEGRATE Mar 1, 2022
Christoph's Tasks automation moved this from TO INTEGRATE to TO REVIEW (max 4 PRs) Mar 1, 2022
The cs-fixer pulls in symfony and friends, which gets autoloader with
the Nextcloud process and causes incompatibilities with Nextcloud's copy
of Symfony. There is no reason to autoload this by default. We can use
the composer bin plugin instead.

Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
@ChristophWurst ChristophWurst merged commit ef78802 into main Mar 3, 2022
Christoph's Tasks automation moved this from TO REVIEW (max 4 PRs) to DONE Mar 3, 2022
@ChristophWurst ChristophWurst deleted the fix/cs-fixer-dep-hell branch March 3, 2022 05:58
jotoeri added a commit to nextcloud/forms that referenced this pull request Apr 6, 2022
Taken from nextcloud/twofactor_webauthn#132

Signed-off-by: Jonas Rittershofer <jotoeri@users.noreply.github.com>
jotoeri added a commit to nextcloud/forms that referenced this pull request Apr 6, 2022
Taken from nextcloud/twofactor_webauthn#132

Signed-off-by: Jonas Rittershofer <jotoeri@users.noreply.github.com>
jotoeri added a commit to nextcloud/forms that referenced this pull request Apr 6, 2022
Taken from nextcloud/twofactor_webauthn#132

Signed-off-by: Jonas Rittershofer <jotoeri@users.noreply.github.com>
jotoeri added a commit to nextcloud/forms that referenced this pull request Apr 6, 2022
Taken from nextcloud/twofactor_webauthn#132

Signed-off-by: Jonas Rittershofer <jotoeri@users.noreply.github.com>
jotoeri added a commit to nextcloud/forms that referenced this pull request Apr 6, 2022
Taken from nextcloud/twofactor_webauthn#132

Signed-off-by: Jonas Rittershofer <jotoeri@users.noreply.github.com>
jotoeri added a commit to nextcloud/forms that referenced this pull request Apr 6, 2022
Taken from nextcloud/twofactor_webauthn#132

Signed-off-by: Jonas Rittershofer <jotoeri@users.noreply.github.com>
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
No open projects
Development

Successfully merging this pull request may close these issues.

Dependency hell due to cs-fixer and symfony console
2 participants