Skip to content

Conversation

@kanongil
Copy link
Contributor

@kanongil kanongil commented Aug 5, 2025

Fortunately it was possible to spot the test issue.

@Marsup
Copy link
Contributor

Marsup commented Aug 5, 2025

Looks like we need a different behavior for node >= 18?

@kanongil kanongil added the test Test or coverage label Aug 6, 2025
@kanongil
Copy link
Contributor Author

kanongil commented Aug 6, 2025

Node 18+ is fixed in my other PR #102 😀

@Marsup
Copy link
Contributor

Marsup commented Aug 6, 2025

OK, let's test that in the other PR then.

@Marsup Marsup merged commit 245850b into hapijs:master Aug 6, 2025
4 of 10 checks passed
@Marsup Marsup added this to the 8.1.1 milestone Aug 6, 2025
@Marsup Marsup self-assigned this Aug 6, 2025
@kanongil kanongil deleted the fix-windows-test branch August 11, 2025 08:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test Test or coverage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants