Skip to content
This repository was archived by the owner on Jul 29, 2024. It is now read-only.

update deps#3

Closed
digitalkaoz wants to merge 1 commit intostackphp:masterfrom
digitalkaoz:patch-1
Closed

update deps#3
digitalkaoz wants to merge 1 commit intostackphp:masterfrom
digitalkaoz:patch-1

Conversation

@digitalkaoz
Copy link

No description provided.

@digitalkaoz
Copy link
Author

@igorw @CHH would you mind to merge this?

"symfony/http-foundation": "~2.1",
"symfony/http-kernel": "~2.1"
"symfony/http-foundation": "~2.1 | ~3.0",
"symfony/http-kernel": "~2.1 | ~3.0"
Copy link
Contributor

Choose a reason for hiding this comment

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

I would update to "~2.7|~3.0" as the other versions are not supported anymore. Also, we need to be sure that this works on both 2.7 and 3.0, so I would also change the .travis.yml file to be sure that we are running tests for at least those two versions. You can have a look at twigphp/twig for an example of how to make it work.

Copy link

Choose a reason for hiding this comment

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

I would then actually forbid 3.0 too.

@fabpot
Copy link
Contributor

fabpot commented Feb 20, 2018

see #4

@fabpot fabpot closed this Feb 20, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants