Skip to content
This repository has been archived by the owner on Jul 24, 2024. It is now read-only.

Test directory is included in published bundle #2739

Closed
votemike opened this issue Sep 18, 2019 · 1 comment
Closed

Test directory is included in published bundle #2739

votemike opened this issue Sep 18, 2019 · 1 comment

Comments

@votemike
Copy link

For some reason the test directory is included in the published files. I would have thought that this makes a yarn/npm install of the bundle slower. https://github.com/sass/node-sass/blob/master/package.json#L44

Can it be removed?
Not sure if all those other files/directories are also needed.
@xzyfer
Copy link
Contributor

xzyfer commented Sep 18, 2019

This is by design. Please search the closed issues for details.

@xzyfer xzyfer closed this as completed Sep 18, 2019
jiongle1 pushed a commit to scantist-ossops-m2/node-sass that referenced this issue Apr 7, 2024
@glebm @xzyfer
Clean up Compound_Selector::operator==
2a4935c 
Implements equality for `Compound_Selector` in a cleaner and more efficient manner.
Similar change for `Selector_List`: sass#2739

This does not fix the stack overflow reported in sass#2667, though it does
change its location (because we no longer clone any nodes in `operator==`).
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@xzyfer @votemike