* Fix Node v25 errors on tests

* Disable fail-fast on test jobs

Co-authored-by: mceachen <216377+mceachen@users.noreply.github.com>

* Revert "Fix Node v25 test errors (#1454)"

This reverts commit 128cdf4.

* ci: disable fail-fast in test job strategy

)

fix: use Holder() instead of This() in PropertyCallbackInfo contexts

PropertyCallbackInfo::This() was removed in V8 13.x (Electron 41),
causing compilation errors in NODE_GETTER functions.

Co-authored-by: mceachen <216377+mceachen@users.noreply.github.com>

* Requires Node.js v20 or later

* generic wording

* link

Co-authored-by: mceachen <216377+mceachen@users.noreply.github.com>

fix: use HolderV2() for PropertyCallbackInfo on V8 >= 13

PR #1456 replaced This() with Holder() in NODE_GETTER contexts, but
Electron 41 (V8 14.6) removed both This() and Holder() from
PropertyCallbackInfo. Only HolderV2() remains (to be renamed back to
Holder() in a future V8 version per crbug.com/333672197).

Add a PROPERTY_HOLDER compat macro that uses HolderV2() on V8 >= 13
and falls back to This() on older versions. Same pattern as the
existing GET_PROTOTYPE macro.

Verified: compiles and runs on Electron 41.0.1 (V8 14.6.202).

Fixes #1458