Skip to content

Conversation

@izaera
Copy link
Member

@izaera izaera commented Jan 9, 2023

Fixes #1049

@bryceosterhaus
Copy link
Member

LGTM!

Any idea why we added that Regexp in the first place?

@izaera
Copy link
Member Author

izaera commented Jan 10, 2023

Any idea why we added that Regexp in the first place?

It had a leading ^ so I guess I wanted to be sure that the SCSS plugin was only applied to files in that directory, but didn't notice that it can be expressed with a simple string.

Webpack configuration is not documented in a very formal way, so sometimes it is difficult to know what can be configured or not and what are the exact semantics behind it 🤷 .

@izaera izaera merged commit 2effd74 into liferay:master Jan 10, 2023
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

Successfully merging this pull request may close these issues.

Liferay/CLI project fails on scss files on Windows

2 participants