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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add bash-installer #352

Merged
merged 3 commits into from
Apr 1, 2024
Merged

Conversation

tucksaun
Copy link
Contributor

Because during his talk @lyrixx mentioned the wish to have one similar to the one from Symfony CLI, here is a PR for a first iteration.

This script is mainly extracted from the Symfony CLI and adapted for 馃Λ Castor.

Can currently be tested using curl "https://raw.githubusercontent.com/tucksaun/castor/feat/bash-installer/installer/bash-installer" | bash

Screenshot 2024-03-29 at 11 05 26 Screenshot 2024-03-29 at 11 05 55

Copy link
Member

@pyrech pyrech left a comment

Choose a reason for hiding this comment

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

This looks really cool, thanks for your contribution \o/

Comment on lines 111 to 113
output " [*] PHP version >= 8.1" "success"
else
output " [ ] ERROR: PHP version is not 8.1 or greater" "error"
Copy link
Member

Choose a reason for hiding this comment

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

Should we display the current detected PHP version (in both success and error cases)? Fe in case the user has several versions installed, it could help to understand the issue. WDYT ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I agree 馃憤
done

doc/getting-started/installation.md Outdated Show resolved Hide resolved
tucksaun and others added 2 commits March 31, 2024 07:00
Co-authored-by: Lo茂ck Piera <pyrech@gmail.com>
@lyrixx lyrixx merged commit 14dad7c into jolicode:main Apr 1, 2024
9 checks passed
@lyrixx
Copy link
Member

lyrixx commented Apr 1, 2024

Thanks you very much

@tucksaun tucksaun deleted the feat/bash-installer branch April 1, 2024 16:40
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.

None yet

3 participants