ci: node nightly test should use node 23

[Uzlopak]

today again i got an email that our nightly fails. checked it and wondered why node nightly from 24 april was used. checked the node page and saw, that since we have node 23, no node 22 nightlies are published. So we have to change the target.

Also here I wonder if the following condition is correct:

undici/.github/workflows/nightly.yml

Line 37 in 38c8b55

if: ${{ always() && (needs.test-linux.result == 'failure' && needs.test-windows.result == 'failure' && needs.test-macos.result == 'failure') }}

I would actually say, that it should be or conditions and not and conditions. If the tests fail in windows but mac and linux pass, as it is now, then we dont get an issue created. Rn i still get an email that it failed.

Maybe @mweberxyz share his opinion?

Maybe can be taken care in a follow up.

[metcoder95]

I remember is was done in this way to reduce false positives as we had them in the past, remember @mweberxyz attempted to fix them.

[Uzlopak]

Ah, ok. Then lets merge this.