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

Support for custom imports + tests for this scenario #20

Merged
merged 1 commit into from
Mar 22, 2015
Merged

Support for custom imports + tests for this scenario #20

merged 1 commit into from
Mar 22, 2015

Conversation

kevin-smets
Copy link

Custom imports will be ignored. the train of thought is as follows:

File not found? -> assuming it's a custom import -> this entry will not be added to the graph.

var errMsg = "File to import not found or unreadable: " + sassPath;
throw errMsg;

var warnMsg = 'File to import not found or unreadable: assuming "' + scssPath +
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For the moment just make this a comment and return false.
I'll add a proper logging interface later.

@xzyfer
Copy link
Owner

xzyfer commented Mar 5, 2015

Once you're done please squash your commits and I'll release this tomorrow as 1.1.0 :)

@kevin-smets
Copy link
Author

Done and done :).

@kevin-smets
Copy link
Author

Is this good to merge?

@xzyfer
Copy link
Owner

xzyfer commented Mar 18, 2015

Hi @kevin-smets can you please rebase this of the current master?

@kevin-smets
Copy link
Author

Done and done @xzyfer

xzyfer added a commit that referenced this pull request Mar 22, 2015
…-compass-imports

Add support for custom imports
@xzyfer xzyfer merged commit 5f312ec into xzyfer:master Mar 22, 2015
@xzyfer
Copy link
Owner

xzyfer commented Mar 22, 2015

Thanks @kevin-smets!

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.

None yet

2 participants