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

Upgrade Platform.sh configuration snippet. #5633

Closed
wants to merge 2 commits into from
Closed

Upgrade Platform.sh configuration snippet. #5633

wants to merge 2 commits into from

Conversation

GuGuss
Copy link

@GuGuss GuGuss commented Aug 18, 2015

toolstack: "php:symfony"

# The type of the application to build.
type: php:5.4
Copy link
Member

Choose a reason for hiding this comment

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

Giving a version number here does work? Just asking as it is not explained in the docs.

Copy link
Member

Choose a reason for hiding this comment

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

if it does, I would vote to use a newer version if possible (although no PHP is used by this project, it's quite nice to show a good practice not using versions that reached eol)

@GuGuss
Copy link
Author

GuGuss commented Aug 19, 2015

This is documented here: https://docs.platform.sh/user_guide/reference/platform-app-yaml.html#type-and-build.

Platform.sh can currently build 3 of the following PHP versions: 5.4, 5.5, 5.6.

Thanks for the review guys!

@wouterj
Copy link
Member

wouterj commented Aug 19, 2015

👍

1 similar comment
@xabbuh
Copy link
Member

xabbuh commented Aug 19, 2015

👍

@xabbuh
Copy link
Member

xabbuh commented Aug 19, 2015

Thank you Augustin.

xabbuh added a commit that referenced this pull request Aug 19, 2015
This PR was submitted for the 2.7 branch but it was merged into the 2.3 branch instead (closes #5633).

Discussion
----------

Upgrade Platform.sh configuration snippet.

Following a recent Platform.sh upgrade: https://docs.platform.sh/user_guide/reference/upgrade/index.html

Commits
-------

df0afdd Upgrade Platform.sh configuration snippet.
@wouterj wouterj closed this Aug 19, 2015
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