- Notifications
You must be signed in to change notification settings - Fork488
Add SASS support#362
base:master
Are you sure you want to change the base?
Add SASS support#362
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Bogala commentedJul 13, 2018
It lacks a dependency to node-sass package. |
thefill commentedJul 13, 2018
@Bogala thanks a lot for pointing this out - will take a look at this. |
thefill commentedJul 13, 2018
@Bogala dependency added - one of the global packages must have fulfilled this dependency. Thanks again! |
thefill commentedJul 13, 2018
@Bogala looks like travis-ci failure is due to to issue with "npm cache clean". Can we execute CI again? |
thefill commentedJul 13, 2018
Apparently npm cache clean should be run with --force. More about this issuehere |
clakech commentedJul 17, 2018
what about node-sass-chokidar ? |
thefill commentedJul 17, 2018
@clakech that is indeed a valid replacement - in fact, its recommended in "react-create-app" doc. Thanks |
Bogala commentedJul 17, 2018
@thefill it is anissue node-sass with npm5. Use node-sass-chokidar will surely allow to pass the CI job |
thefill commentedJul 17, 2018
thefill commentedJul 20, 2018
Any chance to merge this pr ? ;-) |
clakech commentedJul 21, 2018 via email
Oh yes pleaseLe sam. 21 juil. 2018 à 01:08, Filip Dabrowski <notifications@github.com> aécrit : Any chance to merge this pr ? ;-) — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#362 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABHXhn0cNWXPjPie6sDjq3Y87hc0ZXHUks5uImMFgaJpZM4VNUBB> . -- Cyril Lakech |
DorianGrey commentedJul 23, 2018
This fork aims to maintain feature parity with CRA as far as this makes sense considering the technical circumstances - so, at first glance: No, it's favored to wait for CRA to provide this. Yet ... there is support for Which means that there are two changes to take care of:
|
kmturley commentedAug 23, 2018
I forked this repo and added sass-loader support: I pushed to npm as: Which means you can start apps using: |
nickserv commentedSep 1, 2018
This is fixed in CRA 2. |
cyril-lakech commentedSep 28, 2018
now CRA2 is here, what are the plan to use CRA2/SASS/TypeScript ? |
johnrom commentedSep 28, 2018 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Hi all, I've created a fork merging@thefill 's changes with the latest Unfortunately, either@thefill 's repository does not accept PRs or Github is bugging out, so if@thefill wants to PR my PR into his PR, that seems like the proper way to do this, otherwise I can PR this repository directly. https://github.com/nimblelight/create-react-app-typescript/tree/thefill/master-2 |
thefill commentedSep 28, 2018
Hey@johnrom its github's issue I'm afraid - have no restrictions from my side ;-/ best if you can PR it directly if you be so kind ;-) btw I abandoned those scripts completely with all its complex webpack configs... for last few weeks my team is using parcel & all is sooooooo much faster / better / easier. If anybody fancy I can create a quick&dirty public repo with a sample setup. |
johnrom commentedOct 1, 2018
ok, then this PR can be closed, I'll make a reference from my PR to here for documentation purposes. I think my team will stick with the |
I have noticed .scss files are not supported in your fork - I have altered configs so css/scss files are compiled via webpack (no need for external sass compiler).