-
-
Notifications
You must be signed in to change notification settings - Fork 94
Conversation
.travis.yml
Outdated
@@ -48,6 +48,8 @@ matrix: | |||
env: SYMFONY=3.3.* | |||
- php: '7.1' | |||
env: SYMFONY='dev-master as 3.3' | |||
- php: 7.1 | |||
env: DEPENDENCIES=beta |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We manage this file through dev-kit, you aren't supposed to touch this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I saw that in the note in this file. @dbu suggested it would be okay to edit this to start off with, to see what breaks, before finally updating it via the dev-kit.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No need to, look at the Travis jobs, there is one for sf 4 (dev-master as 3.3)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll just remove this then :)
I think we shouldn't merge until we know we are compatible. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please update the PR message body.
Please confirm that everything will work on symfony 4
Could you please rebase your PR and fix merge conflicts? |
fixed in #288 |
I am targeting this branch, because {reason}.
Closes #{put_issue_number_here}
Changelog
To do
Subject