Skip to content

test: ensure compatibility with AsyncLocalStorage #1331

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

Draft
wants to merge 2 commits into
base: v3
Choose a base branch
from

Conversation

bjohansebas
Copy link
Member

related to #1111, adding this test doesn't hurt

@bjohansebas bjohansebas force-pushed the test-asl branch 3 times, most recently from e74a002 to edfb818 Compare June 2, 2025 23:10
@bjohansebas
Copy link
Member Author

Interesting how it fails on Node 18 on macos

@bjohansebas bjohansebas force-pushed the test-asl branch 2 times, most recently from 4755ed0 to a3cef99 Compare June 3, 2025 19:12
Signed-off-by: Sebastian Beltran <[email protected]>
@bjohansebas bjohansebas marked this pull request as draft June 3, 2025 19:58
@coveralls
Copy link

coveralls commented Jun 4, 2025

Coverage Status

coverage: 100.0%. remained the same
when pulling 617a6a0 on bjohansebas:test-asl
into 7dd1e55 on expressjs:v3.

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