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

Updates local #53

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Updates local #53

wants to merge 1 commit into from

Conversation

gilzow
Copy link
Contributor

@gilzow gilzow commented Jul 20, 2023

Description

REsyncs psh config files in this repo with template builder
reorders properties in psh config files to match docs
adds inline docs to psh config files
adds github workflows
renames upstream readme to README_UPSTREAM

All other changes are from the laravel upstream. @thomasdiluccio can you look through those changes and make sure everything is 👍 ?

@gilzow gilzow self-assigned this Jul 20, 2023
@github-actions
Copy link

Visual Regression Testing failed. Please view the report artifact at https://github.com/platformsh-templates/laravel/actions/runs/5616601483#artifacts

@gilzow
Copy link
Contributor Author

gilzow commented Jul 20, 2023

looks like the icon set and font has changed in the most recent version

@gilzow
Copy link
Contributor Author

gilzow commented Jul 27, 2023

v9.52.12 of laravel includes dependencies that are incompatible with php v8.0 and now require v8.1 or greater. bumped the php version in the config file to 8.1

@github-actions
Copy link

Visual Regression Testing failed. Please view the report artifact at https://github.com/platformsh-templates/laravel/actions/runs/5683979201#artifacts

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