Skip to content

Conversation

@Nyholm
Copy link
Member

@Nyholm Nyholm commented Mar 21, 2021

This make sure we dont break things accidentally.

@GrahamCampbell
Copy link
Member

Why not make this the same checks.yml as in guzzle/guzzle?

@Nyholm
Copy link
Member Author

Nyholm commented Mar 21, 2021

  1. It doesn't really make a difference.
  2. That would be a merge conflict when merging up 1.x to master.

I didnt think of it more than that.

@GrahamCampbell GrahamCampbell added this to the 1.8.0 milestone Mar 21, 2021
@Nyholm Nyholm merged commit d963ab3 into guzzle:1.x Mar 21, 2021
@Nyholm Nyholm deleted the bc branch March 21, 2021 12:15
Nyholm added a commit that referenced this pull request Mar 21, 2021
* Soft-final all classes that are final in 2.x (#385)

* Adding integration tests (#390)

* fix: seek stream cloned from php input to begining (#382)

* fix: seek stream cloned from php input to begining

* Update CI

Co-authored-by: Nyholm <tobias.nyholm@gmail.com>

* Locale independent URL parsing (#386)

Co-Authored-By: Alexey Shokov <alexey@shockov.com>

Co-authored-by: Alexey Shokov <alexey@shockov.com>

* Adding tests for BC checks (#393)

* Prepare release 1.8.0 (#394)

* Prepare release 1.8.0

* Update CHANGELOG.md

* Remove period

* Fixes post-merge

Co-authored-by: Tobias Nyholm <tobias.nyholm@gmail.com>
Co-authored-by: Aleh Hutnikau <hutnikau@1pt.com>
Co-authored-by: Alexey Shokov <alexey@shockov.com>
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.

2 participants