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

Allow include in standalone #82

Merged
merged 1 commit into from
Oct 28, 2013
Merged

Allow include in standalone #82

merged 1 commit into from
Oct 28, 2013

Conversation

dn
Copy link
Contributor

@dn dn commented Oct 26, 2013

I tried to run

haml-coffee -r

+include 'partial.haml'

I got a 'browser is not defined'. Changing the condition like below solved the problem for me.

@@ -37,7 +37,7 @@ module.exports = class Directive extends Node
when 'global' then "#{ @namespace }['#{ name }'].apply(#{ context })"
when 'amd' then "require('#{ name }').apply(#{ context })"
when 'standalone'
if browser.process
if browser? && browser.process
Copy link
Collaborator

Choose a reason for hiding this comment

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

if browser?.process

@mehcode
Copy link
Collaborator

mehcode commented Oct 26, 2013

Thanks for the fix.

@netzpirat netzpirat merged commit 196f047 into netzpirat:master Oct 28, 2013
@netzpirat
Copy link
Owner

Thanks! Version 1.13.4 released.

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.

3 participants