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

Explicit composer dependency of php-xml #2515

Closed
wants to merge 3 commits into from

Conversation

zoiosilva
Copy link

Resolves #2377, Resolves #434

It's similar to PR #2010, but this PR merges to 3.x instead of 2.x.

@zoiosilva zoiosilva requested a review from a team as a code owner November 17, 2023 00:31
@zoiosilva zoiosilva changed the title Explicit composer dependency to php-xml Explicit composer dependency of php-xml Nov 17, 2023
@namespacebrian
Copy link
Contributor

Copy/paste from issue #2377:

@zoiosilva Note that installing terminus using composer is not supported. It's strongly recommended to download the .phar from the github releases page and place it in your $PATH, or for users not comfortable doing that, using the brew or ubuntu package.

I think we'll merge the PR for this because php-xml is indeed needed, but this shouldn't be misconstrued as supporting installing terminus using composer.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants