Skip to content

fix(cli-repl): fix lint error#2728

Merged
addaleax merged 1 commit into
mainfrom
fix-check-may26
May 8, 2026
Merged

fix(cli-repl): fix lint error#2728
addaleax merged 1 commit into
mainfrom
fix-check-may26

Conversation

@addaleax
Copy link
Copy Markdown
Collaborator

@addaleax addaleax commented May 8, 2026

Fixes Async arrow function has no 'await' expression from 88ede3e.

Fixes `Async arrow function has no 'await' expression` from 88ede3e.
Copilot AI review requested due to automatic review settings May 8, 2026 15:11
@addaleax addaleax requested a review from a team as a code owner May 8, 2026 15:11
@addaleax addaleax added the no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion) label May 8, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR resolves a lint error in UpdateNotificationManager by replacing a placeholder async fetch implementation (that had no await) with a non-async function that returns a rejected Promise when no fetch is provided.

Changes:

  • Replaced an async default fetch stub with Promise.reject(...) to satisfy the linter rule “Async arrow function has no 'await' expression”.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@addaleax addaleax merged commit 6fea3de into main May 8, 2026
18 of 20 checks passed
@addaleax addaleax deleted the fix-check-may26 branch May 8, 2026 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants