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

Remove Symfony #1120

Closed
wants to merge 3 commits into
base: master
from

Conversation

Projects
None yet
10 participants
@fabpot
Contributor

fabpot commented Nov 20, 2018

No description provided.

@drupol

This comment has been minimized.

drupol commented Nov 20, 2018

Why this change?

@Jan0707

This comment has been minimized.

Jan0707 commented Nov 20, 2018

What's the reasoning here?

@javiereguiluz

This comment has been minimized.

javiereguiluz commented Nov 20, 2018

@drupol @Jan0707 Fabien explained the reasons in this Twitter thread: https://twitter.com/fabpot/status/1064946698089365505

@mickaelandrieu

This comment has been minimized.

mickaelandrieu commented Nov 20, 2018

I don't understand you here @fabpot: in which case(s) removing Symfony project from PHP Fig will help the interoperability and PHP standards in the future?

Do we want to come back 6 years ago with custom autoloading, custom coding styles and custom ways to do everything ?

@Pierstoval

This comment has been minimized.

Pierstoval commented Nov 20, 2018

Do we want to come back 6 years ago with custom autoloading, custom coding styles and custom ways to do everything ?

Is this really about autoloading and coding style? Because Fabien's thread is clear about this: they both are great PSRs, it's mostly the "new wave of incoming PSRs" that's not interop and kinda breaks the initial project into a more opinionated set of rules instead of an interop one

@prolic

prolic approved these changes Nov 20, 2018

@Bilge

This comment has been minimized.

Bilge commented Nov 20, 2018

@fabpot is completely correct, on all points. And to his point about PSR-7 specifically, there's actually an inherent security flaw in its design pertaining to file uploads. I tried to alert the FIG to it some years ago, but given that they deliberately hide the GitHub issues feature from the repository, I took it as a strong indicator that they don't want to know.

@php-fig php-fig locked as too heated and limited conversation to collaborators Nov 20, 2018

@michaelcullum

This comment has been minimized.

Member

michaelcullum commented Nov 20, 2018

This isn't a place for this discussion. Please use the mailing list.

Jean85 added some commits Dec 10, 2018

@michaelcullum

This comment has been minimized.

Member

michaelcullum commented Dec 10, 2018

Merged in 2e54756 but github seems to be having issues

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.