Add then and catch methods to Promise class

[miguelmarcondesf]

Issue: 1667

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 53.84615% with 18 lines in your changes missing coverage. Please review.

Project coverage is 65.58%. Comparing base (0b70d4d) to head (e3dd851).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
napi-inl.h	53.84%	3 Missing and 15 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1668      +/-   ##
==========================================
- Coverage   65.79%   65.58%   -0.22%     
==========================================
  Files           3        3              
  Lines        2146     2185      +39     
  Branches      715      730      +15     
==========================================
+ Hits         1412     1433      +21     
- Misses        162      165       +3     
- Partials      572      587      +15     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[legendecas]

The CI failures are caused by missing of the definition of Promise::Promise().

[miguelmarcondesf]

The CI failures are caused by missing of the definition of Promise::Promise().

Thank you, just fixed that!

[KevinEady]

Mostly some changes on the docs... but the implementation looks good! Thanks for the contribution 👍

[KevinEady]

LGTM but some small nits / questions.

Note the the CI will fail until #1672 is merged and this branch is rebased.

[miguelmarcondesf]

LGTM but some small nits / questions.

Note the the CI will fail until #1672 is merged and this branch is rebased.

Sounds good, thanks!

[mhdawson]

LGTM