Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings
This repository was archived by the owner on Jul 24, 2024. It is now read-only.

remove mkdirp dep#3108

Merged
xzyfer merged 2 commits into
sass:mastersass/node-sass:masterfrom
jimmywarting:remove-mkdirpjimmywarting/node-sass:remove-mkdirpCopy head branch name to clipboard
Jun 24, 2021
Merged

remove mkdirp dep#3108
xzyfer merged 2 commits into
sass:mastersass/node-sass:masterfrom
jimmywarting:remove-mkdirpjimmywarting/node-sass:remove-mkdirpCopy head branch name to clipboard

Conversation

@jimmywarting

Copy link
Copy Markdown
Contributor

closes #3003

@xzyfer

xzyfer commented May 17, 2021

Copy link
Copy Markdown
Contributor

Thanks @jimmywarting. I'm happy to ship this if CI is happy.

@xzyfer

xzyfer commented May 17, 2021

Copy link
Copy Markdown
Contributor

Pretty sure these failures are expected.

@jimmywarting

jimmywarting commented May 25, 2021

Copy link
Copy Markdown
Contributor Author

Not entierly sure what failed with Build bindings for Alpine releases / build (16)

@nschonni nschonni left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think the Alpine thing is unrelated. Not sure but I remember it being flakey initially after the previous major, and the current one anyway

@xzyfer
xzyfer merged commit 911d4db into sass:master Jun 24, 2021
@jimmywarting
jimmywarting deleted the remove-mkdirp branch June 24, 2021 08:23
@xzyfer

xzyfer commented Jun 24, 2021

Copy link
Copy Markdown
Contributor

Appreciate all your effort. This is released in 6.0.1.

@jimmywarting

Copy link
Copy Markdown
Contributor Author

Np.

Do you want me to make a PR that uses Node's new fs.rm(path, { recursive }) instead of the devDep rimraf too?

@xzyfer

xzyfer commented Jun 24, 2021

Copy link
Copy Markdown
Contributor

Totally up to you. It's not something we plan to do but we're happy to look at a PR :)

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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.