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

tweaks so compass-core assets are detected and included in load path #83

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

srerickson
Copy link

Not sure if this is the correct approach, but it gets compass working for me

@tdreyno
Copy link
Member

tdreyno commented Nov 24, 2015

Awesome. I was going to do a sprint on this extension this weekend. I'll confirm this works then and merge. Thanks!

@srerickson
Copy link
Author

Actually, after more thinking, I'm not sure this change makes sense: append_paths_from_gems will detect assets in gems that might declare themselves, causing duplicate entries in the load path. For example, susy registers itself (for some reason Compass is not defined when that happens for me, so it uses SASS_PATH); so append_paths_from_gems adds it to the load path a second time
... still it's strange that compass's self-registration doesn't work and we need to grab it this way.

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.

2 participants