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

Will you add commonjs support in the npm package @connorskees/grass? #74

Open
Priestch opened this issue Apr 22, 2023 · 2 comments
Open

Comments

@Priestch
Copy link

I try to replace node-sass in a project, but found only the ES module is supported, will you consider supporting this?

@connorskees
Copy link
Owner

connorskees commented Jun 30, 2023

I am open to supporting this, but I will likely not implement anything myself until I have a usecase for it.

I'm not aware of any benefits to using grass as a replacement for node-sass compared to the official sass package. I have not benchmarked a potential implementation of a node package using FFI, and I am not interested in maintaining a node API.

Is there a reason you would like commonjs to be supported, other than potential performance gains?

@Priestch
Copy link
Author

All I want are the potential performance gains.

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

No branches or pull requests

2 participants