Skip to content

Handle promise rejection in demoGithubUser call#3965

Open
AlexNT-maker wants to merge 2 commits into
javascript-tutorial:masterfrom
AlexNT-maker:master
Open

Handle promise rejection in demoGithubUser call#3965
AlexNT-maker wants to merge 2 commits into
javascript-tutorial:masterfrom
AlexNT-maker:master

Conversation

@AlexNT-maker

Copy link
Copy Markdown

Added a .catch(alert) safety net to the demoGithubUser() call to prevent an unhandled promise rejection in case the thrown error is not an HttpError.

Added a .catch(alert) safety net to the demoGithubUser() call to prevent an unhandled promise rejection in case the thrown error is not an HttpError.
@javascript-translate-bot javascript-translate-bot added the review needed Review needed, please approve or request changes label May 22, 2026
@javascript-translate-bot
javascript-translate-bot requested a review from a team May 22, 2026 15:23
@smith558

Copy link
Copy Markdown
Member

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Keep them coming!

Reviewed commit: 705abf81f7

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review needed Review needed, please approve or request changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants