Skip to content
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

chore(docs): remove leading dollar sign #95

Merged
merged 1 commit into from Jun 24, 2022
Merged

chore(docs): remove leading dollar sign #95

merged 1 commit into from Jun 24, 2022

Conversation

nhedger
Copy link
Contributor

@nhedger nhedger commented Jun 20, 2022

README.md Outdated Show resolved Hide resolved
@clue clue added this to the v0.6.5 milestone Jun 20, 2022
@clue
Copy link
Member

clue commented Jun 20, 2022

The build error is unrelated to this changeset and is actually caused by an upstream regression in PHP 8.1.7 and PHP 8.0.20 only (see php/php-src#8575, php/php-src#8571 and php/php-src#8827). There's ongoing effort to revert this for upcoming PHP versions again, so the build should be green again by then, no changes required here.

@clue
Copy link
Member

clue commented Jun 21, 2022

The build error is unrelated to this changeset and is actually caused by an upstream regression in PHP 8.1.7 and PHP 8.0.20 only (see php/php-src#8575, php/php-src#8571 and php/php-src#8827). There's ongoing effort to revert this for upcoming PHP versions again, so the build should be green again by then, no changes required here.

See #96 for temporary test adjustments to skip affected tests.

Copy link
Member

@clue clue left a comment

Choose a reason for hiding this comment

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

@nhedger Thanks for the quick update, changes LGTM! 👍

@clue clue requested a review from WyriHaximus June 21, 2022 20:29
@WyriHaximus WyriHaximus merged commit 127eadc into reactphp:0.6.x Jun 24, 2022
@nhedger nhedger deleted the chore/remove-leading-dollar branch June 24, 2022 09:58
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.

None yet

3 participants