Skip to content

Moved Symfony before Buzz #141

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 3 commits into from
Jun 27, 2019
Merged

Moved Symfony before Buzz #141

merged 3 commits into from
Jun 27, 2019

Conversation

Nyholm
Copy link
Member

@Nyholm Nyholm commented Jun 24, 2019

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Related tickets
Documentation
License MIT

I moved Symfony before Buzz since it is fairly more used and maintained. Im not sure if this list should be ordered by popularity, performance, most likeable or random. But this feels better.

@Nyholm
Copy link
Member Author

Nyholm commented Jun 26, 2019

Should we update order on the HTTPlug adapters too? @dbu @sagikazarmark?

@sagikazarmark
Copy link
Member

I'm okay if we suggest a better maintained client. It only matters anyway if there are multiple ones installed.

For the user it shouldn't really matter, they should rely on the interface. The only thing that would impact them is a client not adhering the standard or performance differences. Again, I think we can make the better choice for the user in those cases as well.

@Nyholm
Copy link
Member Author

Nyholm commented Jun 26, 2019

Okey. Thank you for the input. I've updated the PR

@Nyholm Nyholm merged commit e444002 into master Jun 27, 2019
@Nyholm
Copy link
Member Author

Nyholm commented Jun 27, 2019

Thank you for the reviews

@Nyholm Nyholm deleted the patch-sf branch June 27, 2019 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants