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

Allow USWDS Compile to have option of Sass or Sass embedded #32

Open
brunerae opened this issue Jun 6, 2022 · 4 comments
Open

Allow USWDS Compile to have option of Sass or Sass embedded #32

brunerae opened this issue Jun 6, 2022 · 4 comments
Assignees

Comments

@brunerae
Copy link

brunerae commented Jun 6, 2022

Relates to: #20 and #28

Create a new setting that lets you compile Sass with plain Sass or Sass embedded.

@thisisdano
Copy link
Member

@mejiaj I'm disinclined to pursue this solution, as it seems like an unnecessary complexity, but I'd like your opinion on the severity of the problem this would solve and the complexity of the solution.

@thisisdano thisisdano moved this from Needs grooming to TBD in USWDS Core Project Data Jun 29, 2022
@smustgrave
Copy link

ran into this error again for ddev same issue.

@smustgrave
Copy link

Updating sass-embeded fixed it https://github.com/uswds/uswds-compile/pull/47/files

@mejiaj
Copy link
Contributor

mejiaj commented Mar 16, 2023

@thisisdano agreed. Sass embedded is meant to support legacy and new API. This doesn't seem like an issue any more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: TBD
Development

No branches or pull requests

4 participants