Skip to content

fix(napi): nested Either and Promise validate logic - #2625

Merged
Brooooooklyn merged 1 commit into
mainfrom
05-11-fix_napi_nested_either_and_promise_validate_logic
May 11, 2025
Merged

Brooooooklyn merged 1 commit into
mainfrom
05-11-fix_napi_nested_either_and_promise_validate_logic

Conversation

@Brooooooklyn

@Brooooooklyn Brooooooklyn commented May 11, 2025 •

Copy link
Copy Markdown
Member

Brooooooklyn commented May 11, 2025 •

Copy link
Copy Markdown
Member Author

Brooooooklyn commented May 11, 2025 •

Copy link
Copy Markdown
Member Author

Merge activity

  • May 11, 12:30 PM EDT: A user started a stack merge that includes this pull request via Graphite.
  • May 11, 12:32 PM EDT: Graphite rebased this pull request as part of a merge.
  • May 11, 12:33 PM EDT: @Brooooooklyn merged this pull request with Graphite.

@Brooooooklyn
Brooooooklyn changed the base branch from 05-11-fix_napi_reference_leak_in_deferredtrace to graphite-base/2625 May 11, 2025 16:31
@Brooooooklyn
Brooooooklyn changed the base branch from graphite-base/2625 to main May 11, 2025 16:31
@Brooooooklyn
Brooooooklyn force-pushed the 05-11-fix_napi_nested_either_and_promise_validate_logic branch from 175c351 to 62dd0b8 Compare May 11, 2025 16:31
@Brooooooklyn
Brooooooklyn merged commit 7a97242 into main May 11, 2025
@Brooooooklyn
Brooooooklyn deleted the 05-11-fix_napi_nested_either_and_promise_validate_logic branch May 11, 2025 16:33
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.

When either::A is a promise, and the either is inside another either, an error occurs

1 participant