Skip to content

Conversation

rpg600
Copy link
Contributor

@rpg600 rpg600 commented Jul 11, 2014

Q A
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? yes
Tests pass? yes
Fixed tickets #10658
License MIT
Doc PR symfony/symfony-docs#3782

@ClementGautier
Copy link
Contributor

@rpg600 Can you squash @snoob second commit with the first one plz ?

@snoob
Copy link
Contributor

snoob commented Jul 16, 2014

I have done your changes on the root PR (#10676). It's useless to create a new one for 3 lines it's avoid duplication and simplify the job for the mergers. If something is still missing just report it on my PR.

@Tobion
Copy link
Contributor

Tobion commented Jul 16, 2014

Since I found some more changes to be done, let's just continue with the original PR that is still active.

@Tobion Tobion closed this Jul 16, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants