Skip to content
This repository was archived by the owner on Jan 6, 2024. It is now read-only.

Conversation

tolry
Copy link

@tolry tolry commented Oct 9, 2018

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

What's in this PR?

see #19

Checklist

  • check if CI completes successfully
  • check with @Nyholm about planned BC breaks before 1.0

@tolry
Copy link
Author

tolry commented Oct 9, 2018

ok, there seem to be actual api breaks with newer version (at least 0.17) - see https://travis-ci.org/php-http/buzz-adapter/jobs/439230909#L525

@Nyholm
Copy link
Member

Nyholm commented Oct 9, 2018

Correct. There are some BC breaks. However. The latest version of Buzz supports HTTPlug. Which means that this adapter is obsolete =)

@dbu
Copy link
Contributor

dbu commented Oct 10, 2018

The latest version of Buzz supports HTTPlug. Which means that this adapter is obsolete =)

ah, haha! but then lets say this in the README. does 0.16 provide httplug already? or is there a gap? i guess either way we should have the readme tell that for 0.15 you can use this and if you are on (whatever supports it) just install buzz. and we could mark this repo as abandoned when the readme is updated, to avoid confusion.

@tolry
Copy link
Author

tolry commented Oct 10, 2018

closing here

@tolry tolry closed this Oct 10, 2018
@tolry tolry deleted the patch-1 branch October 10, 2018 09:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

adapter currently only supports buzz 0.15.*, not 0.16 or 0.17
3 participants