Skip to content

Add Symfony installation support#599

Closed
abdellahrk wants to merge 4 commits into
basecamp:devfrom
abdellahrk:dev
Closed

Add Symfony installation support#599
abdellahrk wants to merge 4 commits into
basecamp:devfrom
abdellahrk:dev

Conversation

@abdellahrk
Copy link
Copy Markdown

Add support for installing Symfony PHP framework:

  • Updated to available commands to include symfony
  • Added symfony) entry in the installation script to install the Symfony CLI

Add support for Symfony PHP framework:
- Added `symfony)` entry in the installation script to install the Symfony CLI
@dhh
Copy link
Copy Markdown
Member

dhh commented Aug 10, 2025

Now that we're installing Laravel via php and composer, we'll need to do the same for this, if it's going to get added in.

@abdellahrk
Copy link
Copy Markdown
Author

Thank you @dhh.
I am adding that

@abdellahrk
Copy link
Copy Markdown
Author

Now that we're installing Laravel via php and composer, we'll need to do the same for this, if it's going to get added in.

It's done. PHP and Composer get install first then Symfony.
Thank you.
@dhh

@dhh
Copy link
Copy Markdown
Member

dhh commented Aug 10, 2025

No, what we need is something like this: https://github.com/basecamp/omarchy/blob/dev/bin/omarchy-install-dev-env#L28-L35

@abdellahrk
Copy link
Copy Markdown
Author

Okay. Noted. I am trying again. Thanks for the review.

@c4software
Copy link
Copy Markdown
Contributor

Hi since the code is mostly the same, I create a new PR with some logic merged, I also used the symfony-cli from AUR instead of curl | bash installation. See the PR #624

@dhh
Copy link
Copy Markdown
Member

dhh commented Aug 10, 2025

I'll credit you both in the release 👍

@dhh dhh closed this Aug 10, 2025
@abdellahrk
Copy link
Copy Markdown
Author

Thank you @c4software and thank you manifold @dhh.

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.

3 participants