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

[FIX] assign correct YML version compatibility to PHP 7.0 #505

Closed
wants to merge 9 commits into from

Conversation

amarroni
Copy link
Contributor

YAML version minimal compatibility changed to PHP 7.1.0, was needed to determine the YAML version will working without problems.
Based on discussion on Channel Slack, the version is 2.0.4 (provide @Neodork thank you.)

Added. that version to 7.0 -> 2.0.4

@Neodork Neodork self-assigned this Sep 21, 2020
@Neodork Neodork added 2.x Issues / features related the the 2.x branch PATCH This change is patch level labels Sep 21, 2020
@Neodork
Copy link
Collaborator

Neodork commented Sep 21, 2020

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@Neodork
Copy link
Collaborator

Neodork commented Sep 22, 2020

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@Neodork Neodork changed the base branch from master to 2.x September 22, 2020 06:36
@Neodork
Copy link
Collaborator

Neodork commented Oct 2, 2020

@amarroni Would you mind rebasing your branch from 2.x? Otherwise I'll have to cherry pick this and you'll lose credits.

@Neodork
Copy link
Collaborator

Neodork commented Oct 2, 2020

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@amarroni
Copy link
Contributor Author

amarroni commented Oct 2, 2020

#541 DONE!

@Neodork
Copy link
Collaborator

Neodork commented Oct 3, 2020

@amarroni Thanks for the effort! Your work is merged and will be included in the next release. I really appreciate the quick response on my request. Thanks again.

You can check your work here: https://github.com/weprovide/valet-plus/commits/2.x

@Neodork Neodork closed this Oct 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2.x Issues / features related the the 2.x branch PATCH This change is patch level
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants