Skip to content

Add support of Symfony 3.0#85

Merged
k-k merged 6 commits intouecode:masterfrom
francoispluchino:sf3-support
Dec 18, 2015
Merged

Add support of Symfony 3.0#85
k-k merged 6 commits intouecode:masterfrom
francoispluchino:sf3-support

Conversation

@francoispluchino
Copy link
Contributor

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets #84
License MIT
Doc PR ~

Copy link
Contributor

Choose a reason for hiding this comment

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

The comment should not start with /** which indicates a docblock.

@xabbuh
Copy link
Contributor

xabbuh commented Dec 11, 2015

There is no job on Travis performing a build with Symfony 3.

@francoispluchino
Copy link
Contributor Author

It's done.

@francoispluchino
Copy link
Contributor Author

Travis test Symfony 3.0 with PHP 5.4. What is the config for testing Symfony 3.0 with PHP 5.5 and greater?

@francoispluchino
Copy link
Contributor Author

The config of Travis is fixed now.

@xabbuh
Copy link
Contributor

xabbuh commented Dec 11, 2015

👍 LGTM

The Travis CI configuration can be improved but that should better be done in a separate pull request.

@francoispluchino
Copy link
Contributor Author

Ok, I'm interested by your optimization of the Travis config.

@xabbuh
Copy link
Contributor

xabbuh commented Dec 11, 2015

@francoispluchino You can take a look at my changes in #86 (one job is currently failing because of a YAML syntax error that will be fixed by #87).

@francoispluchino
Copy link
Contributor Author

@xabbuh Thanks!

composer.json Outdated
Copy link
Contributor

Choose a reason for hiding this comment

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

while you are in here, change the ~'s to ^'s

Copy link
Contributor Author

Choose a reason for hiding this comment

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

^3.0.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 actually say ^3.0, but you could change the other ~'s too

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok I do this.

@scrutinizer-notifier
Copy link

The inspection completed: 26 Issues, 0 Patches

@francoispluchino
Copy link
Contributor Author

It's ok for you now?

@cryptiklemur
Copy link
Contributor

@kmfk

@k-k k-k merged commit 17c16d5 into uecode:master Dec 18, 2015
@k-k
Copy link
Contributor

k-k commented Dec 18, 2015

@francoispluchino Thanks!

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.

5 participants