Skip to content

[3.0] Removed unneeded polyfill #16637

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from Nov 23, 2015
Merged

[3.0] Removed unneeded polyfill #16637

merged 1 commit into from Nov 23, 2015

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