Skip to content
This repository has been archived by the owner on Feb 9, 2023. It is now read-only.

manage all the jade.compile options #7

Closed
wants to merge 2 commits into from
Closed

Conversation

ilkosta
Copy link

@ilkosta ilkosta commented Mar 13, 2013

Hi,
some user requested the support of locals and all compiler options in static-jade-brunch, so I changed the plugin options.

It would be nice if jade-brunch and static-jade-brunch can share the same options.

jade:
options: # can be added all the supported jade options
pretty: yes # Adds pretty-indentation whitespaces to output (false by default)
compiler: funny # Compiler to replace jade's default
Copy link
Contributor

Choose a reason for hiding this comment

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

i don’t see where this exact option is used

Copy link
Author

Choose a reason for hiding this comment

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

considering the user request to customize the jade options and looking at CSRF example I believe it can be useful to allow the passage of all the options available in jade, including that of the compiler.

@paulmillr
Copy link
Contributor

the PR was outdated soon after it appeared, sorry that I ain’t commented on that. needs rebasing

@paulmillr paulmillr closed this May 14, 2013
paulmillr added a commit that referenced this pull request Feb 5, 2014
Added: options can be passed to Jade (from #7)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants