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

Preprocess rails assets imported with extensions. #137

Open
wants to merge 1 commit into
base: master
from

Conversation

@bodnarbm
Copy link

commented Oct 5, 2019

If a processed file imported an erb asset with the erb extension in the import
argument, these files would not be preprocessed by the sassc-rails and the
native sassc functions would fail to compile the assets.

This change checks if the import argument ends with any of the extensions known
by the sassc preprocessor, and if it does, applies the appropriate
preprocessing to that file.

Addresses #123 second issue

If a processed file imported an erb asset with the erb extension in the import
argument, these files would not be preprocessed by the sassc-rails and the
native sassc functions would fail to compile the assets.

This change checks if the import argument ends with any of the extensions known
by the sassc preprocessor, and if it does, applies the appropriate
preprocessing to that file.
@bodnarbm bodnarbm force-pushed the bodnarbm:fix-import-erb-with-extension branch from f53efea to 7e43178 Oct 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.