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

Update close handler to avoid unhandled promise rejections #71

Merged
merged 1 commit into from
Jan 1, 2025

Conversation

clue
Copy link
Owner

@clue clue commented Jan 1, 2025

Builds on top of reactphp/promise#248 and #63

@clue clue added bug Something isn't working new feature New feature or request labels Jan 1, 2025
@clue clue added this to the v1.7.0 milestone Jan 1, 2025
@clue clue force-pushed the unhandled-rejections branch from 6070b21 to 1cd706e Compare January 1, 2025 18:39
@clue clue force-pushed the unhandled-rejections branch from 1cd706e to c15c700 Compare January 1, 2025 18:42
@clue
Copy link
Owner Author

clue commented Jan 1, 2025

Fixed unrelated build error by updating test environment to disable JIT due to Xdebug conflict (refs #64): https://github.com/shivammathur/setup-php#jit-configuration & https://xdebug.org/docs/compat#compat

@clue clue merged commit 9f54bcd into clue:1.x Jan 1, 2025
21 checks passed
@clue clue deleted the unhandled-rejections branch January 1, 2025 18:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working new feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant