Visitar URL original
remove mkdirp dep by jimmywarting · Pull Request #3108 · sass/node-sass · GitHub
Skip to content
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:masterfrom
jimmywarting:remove-mkdirp
Jun 24, 2021
Merged

xzyfer merged 2 commits into
sass:masterfrom
jimmywarting:remove-mkdirp

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