-
Notifications
You must be signed in to change notification settings - Fork 6.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
content: add promises example to accepting input from the command line #7252
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Benson Isaac <[email protected]>
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have a few concerns:
- There is no indication that this is experimental
- This isn't formatted (I.e. spacing and capitalization)
I think, you can use code tabs to make this look cleaner
rl.close() | ||
|
||
console.log(`Hi ${name}!`); | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
rl.close() | |
console.log(`Hi ${name}!`); | |
console.log(`Hi ${name}!`); | |
rl.close() |
or you could use a promise | ||
```mjs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
or you could use a promise | |
```mjs | |
```mjs displayName="Promises API" |
And change the other one accordingly
Description
Validation
Related Issues
Check List
npm run format
to ensure the code follows the style guide.npm run test
to check if all tests are passing.npx turbo build
to check if the website builds without errors.