Skip to content

docs: Replace mkdirp with fsPromises.mkdir in Puppeteer.md#11951

Merged
SimenB merged 12 commits intojestjs:mainfrom
munierujp:patch-1
Oct 13, 2021
Merged

docs: Replace mkdirp with fsPromises.mkdir in Puppeteer.md#11951
SimenB merged 12 commits intojestjs:mainfrom
munierujp:patch-1

Conversation

@munierujp
Copy link
Copy Markdown
Contributor

@munierujp munierujp commented Oct 12, 2021

Summary

Replce mkdirp with fsPromises.mkdir in Puppeteer.md

Test plan

This is a document-only change.

See also

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #11951 (96e549c) into main (3a85065) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main   #11951   +/-   ##
=======================================
  Coverage   68.76%   68.76%           
=======================================
  Files         322      322           
  Lines       16620    16620           
  Branches     4795     4795           
=======================================
  Hits        11429    11429           
  Misses       5159     5159           
  Partials       32       32           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3a85065...96e549c. Read the comment docs.

Copy link
Copy Markdown
Member

@SimenB SimenB left a comment

Choose a reason for hiding this comment

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

thanks! lefta. suggestion I think makes it even cleaner, but cleaning this up is nice 🙂

Comment thread docs/Puppeteer.md Outdated
@munierujp munierujp changed the title docs: Use async API of mkdirp in Puppeteer.md docs: Replce mkdirp with fsPromises.mkdir in Puppeteer.md Oct 13, 2021
Copy link
Copy Markdown
Member

@SimenB SimenB left a comment

Choose a reason for hiding this comment

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

thanks!

@SimenB SimenB merged commit b439692 into jestjs:main Oct 13, 2021
@munierujp munierujp deleted the patch-1 branch October 13, 2021 11:25
@munierujp munierujp changed the title docs: Replce mkdirp with fsPromises.mkdir in Puppeteer.md docs: Replace mkdirp with fsPromises.mkdir in Puppeteer.md Oct 13, 2021
@github-actions
Copy link
Copy Markdown

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.
Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Nov 13, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants