Skip to content
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

replace unnecessary callback functions with async/await #4555

Merged
merged 4 commits into from
Apr 24, 2022
Merged

replace unnecessary callback functions with async/await #4555

merged 4 commits into from
Apr 24, 2022

Conversation

SEWeiTung
Copy link
Contributor

@SEWeiTung SEWeiTung commented Apr 20, 2022

  1. Some functions like 'fs' or 'graceFulFs' has been supporting
    'async/await' for a long time, so this is a refactor to improve
    the related codes and performance when building.

  2. 'sass.render' is deprecated, use awaitable 'sass.compileAsync'
    instead.

@SEWeiTung SEWeiTung changed the title replace necessary async functions with async/await replace unnecessary async functions with async/await Apr 20, 2022
@SEWeiTung SEWeiTung changed the title replace unnecessary async functions with async/await replace unnecessary callback functions with async/await Apr 20, 2022
Maledong and others added 2 commits April 21, 2022 10:01
1. Some functions like 'fs' or 'graceFulFs' has been supporting
'async/await' for a long time, so this is a refactor to improve
the related codes.

2. 'sass.render' is deprecated, use awaitable 'sass.compileAsync'
instead.
@SEWeiTung SEWeiTung requested a review from a team April 21, 2022 07:52
@SEWeiTung SEWeiTung merged commit 0c7d743 into nodejs:main Apr 24, 2022
@SEWeiTung SEWeiTung deleted the improveCode branch April 24, 2022 05:53
oszwilk added a commit to oszwilk/nodejs.org that referenced this pull request Apr 24, 2022
* Blog: Node.js 18 release announcement

Co-authored-by: Antoine du Hamel <[email protected]>
Co-authored-by: Ethan Arrowood <[email protected]>
Co-authored-by: Matteo Collina <[email protected]>
Co-authored-by: Michael Dawson <[email protected]>
Co-authored-by: Michaël Zasso <[email protected]>
Co-authored-by: Rich Trott <[email protected]>
Co-authored-by: Richard Lau <[email protected]>
Co-authored-by: Joyee Cheung <[email protected]>

* Blog: v18.0.0 release post

Refs: nodejs/node#42262

* fix: add Node.js 16.x and 17.x to generated version link list (nodejs#4545)

* fix: add Node.js 16.x and 17.x to generated version link list

`source/nodejsVersions.md` is used to generate a list of version links
on each locale page. Node.js 17.x were missing.

Co-authored-by: Nick Schonning <[email protected]>

* zh-CN: update site.json for Node.js 18 (nodejs#4554)

Ref: nodejs#4553

* Fix typo (nodejs#4557)

"to be trust" should be "to trust" as a transitive verb.

* doc: update link about class features optimizations in Node.js 18 (nodejs#4556)

There is a blog post explaining about this now, change the link.

* doc: typo fix (nodejs#4558)

Add missing "to" between the two verbs as a typo fix.

* doc: fix wrong link for "windows" (nodejs#4514)

The anchor to Windows installation session now points to "install nvs on Windows". This PR fixes the anchor to the correct location.

* chore: replace callbacks with necessary async/await (nodejs#4555)

1. Some functions like 'fs' or 'graceFulFs' has been supporting
'async/await' for a long time, so this is a refactor to improve
the related codes.

2. 'sass.render' is deprecated, use awaitable 'sass.compileAsync'
instead.

* Remove call to deprecated function util.inherits (nodejs#4483)

* Remove call to deprecated function util.inherits

* Update event-loop-timers-and-nexttick.md

Co-authored-by: Rich Trott <[email protected]>

* Meetup sweden stockholm (nodejs#4407)

Add the meetup for sweden stockholm.

Co-authored-by: Beth Griggs <[email protected]>
Co-authored-by: Antoine du Hamel <[email protected]>
Co-authored-by: Ethan Arrowood <[email protected]>
Co-authored-by: Matteo Collina <[email protected]>
Co-authored-by: Michael Dawson <[email protected]>
Co-authored-by: Michaël Zasso <[email protected]>
Co-authored-by: Rich Trott <[email protected]>
Co-authored-by: Richard Lau <[email protected]>
Co-authored-by: Joyee Cheung <[email protected]>
Co-authored-by: Austin Kelleher <[email protected]>
Co-authored-by: Nick Schonning <[email protected]>
Co-authored-by: Maledong <[email protected]>
Co-authored-by: Kenny <[email protected]>
Co-authored-by: Javad <[email protected]>
Co-authored-by: Tse Kit Yam <[email protected]>
Co-authored-by: Bart Louwers <[email protected]>
Co-authored-by: lxmarinkovic <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants