search

Agoric / documentation

User documentation
https://agoric.com/documentation/
Apache License 2.0
15 stars 39 forks source link

doc(orch): use `try-catch`, not `.then` and `.catch` #1158

Closed michaelfig closed 4 months ago

michaelfig commented 4 months ago

Mixing await style with .then and .catch is sufficiently confusing to novice JS programmers. Please take advantage of try-catch instead.

cloudflare-workers-and-pages[bot] commented 4 months ago

Deploying documentation with  Cloudflare Pages  Cloudflare Pages

Latest commit: 279a40a
Status: ✅  Deploy successful!
Preview URL: https://e88f7d4f.documentation-7tp.pages.dev
Branch Preview URL: https://mfig-orch-try-catch.documentation-7tp.pages.dev

View logs

github-actions[bot] commented 4 months ago

Cloudflare deployment logs are available here

michaelfig commented 4 months ago

I had this same thought when looking at the recent vow when retry logic. Would this same guidance still apply there?

Absolutely! Here's the PR that I just now tagged you in: Agoric/agoric-sdk#9731