Skip to content

Commit

Permalink
fix(@angular/cli): wrap opn in a new Promise because opn is alrea…
Browse files Browse the repository at this point in the history
…dy resolve is already resolve.
  • Loading branch information
wKoza authored and alexeagle committed Feb 9, 2019
1 parent c09ebc0 commit 419738c
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions packages/angular/cli/commands/doc-impl.ts
Expand Up @@ -24,8 +24,11 @@ export class DocCommand extends Command<DocCommandSchema> {
searchUrl = `https://www.google.com/search?q=site%3Aangular.io+${options.keyword}`;
}

return opn(searchUrl, {
wait: false,
// We should wrap `opn` in a new Promise because `opn` is already resolved
await new Promise(() => {
opn(searchUrl, {
wait: false,
});
});
}
}

0 comments on commit 419738c

Please sign in to comment.