Skip to content

Conversation

@GrahamCampbell
Copy link
Contributor

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

Surely we don't need these on 3.x?

@stof
Copy link
Member

stof commented Nov 23, 2015

👍

@GrahamCampbell
Copy link
Contributor Author

I also need to sort this out in all the other composer.json files, which was hard to do from the GitHub web interface, so I'll pull this down to my machine in a bit and sort that.

@GrahamCampbell
Copy link
Contributor Author

Actually, now that I've pulled it down, I see that stuff has already been dealt with, and it's just the main composer.json file that needs updating. ;)

@xabbuh
Copy link
Member

xabbuh commented Nov 23, 2015

👍

@Tobion
Copy link
Contributor

Tobion commented Nov 23, 2015

Thank you @GrahamCampbell.

@Tobion Tobion merged commit 42c9a48 into symfony:master Nov 23, 2015
Tobion added a commit that referenced this pull request Nov 23, 2015
This PR was merged into the 3.0-dev branch.

Discussion
----------

[3.0] Removed unneeded polyfill

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | N/A
| License       | MIT
| Doc PR        | N/A

Surely we don't need these on 3.x?

Commits
-------

42c9a48 Removed unneeded polyfill from the master
@GrahamCampbell GrahamCampbell deleted the patch-1 branch November 23, 2015 16:15
@GrahamCampbell
Copy link
Contributor Author

:)

@fabpot fabpot mentioned this pull request Nov 30, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants