Skip to content

Forward compatibility with stream 1.0, 0.7, 0.6, and 0.5 - #2

Merged
WyriHaximus merged 3 commits into
reactphp:masterfrom
WyriHaximus-labs:forward-compatibility
May 14, 2017
Merged

Forward compatibility with stream 1.0, 0.7, 0.6, and 0.5#2
WyriHaximus merged 3 commits into
reactphp:masterfrom
WyriHaximus-labs:forward-compatibility

Conversation

@WyriHaximus

Copy link
Copy Markdown
Member

As mentioned on the roadmap in #1, support the latest stream tags as well.

@WyriHaximus WyriHaximus added this to the v0.2.0 milestone May 13, 2017
@WyriHaximus
WyriHaximus requested review from clue and jsor May 13, 2017 22:01
@clue

clue commented May 13, 2017

Copy link
Copy Markdown
Member

FWIW: This does not affect BC, so we may as well release this as v0.1.1 👍

@WyriHaximus

Copy link
Copy Markdown
Member Author

FWIW: This does not affect BC, so we may as well release this as v0.1.1 👍

I was anticipating more breaking to be honest 🤓, 0.1.1 makes sense to me, what about you @jsor?

Comment thread composer.json
"require": {
"php": ">=5.3",
"react/stream": "^0.4 || ^0.3",
"react/stream": "^1.0 || ^0.7 || ^0.6 || ^0.5 || ^0.4 || ^0.3",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wouldn't include 1.0 here, as 1.0 isn't released yet. Changes could happen that are not planned yet.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Anything between 0.7 and 1.0 will be bug fixes. Nothing major will change

@jsor

jsor commented May 14, 2017

Copy link
Copy Markdown
Member

I'm good with either.

@WyriHaximus
WyriHaximus merged commit 82e0919 into reactphp:master May 14, 2017
@WyriHaximus

Copy link
Copy Markdown
Member Author

Will tag v0.1.1 tomorrow morning 👍

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.

4 participants