Skip to content

Added process name/namespace option for compiling to client#157

Closed
magnuspaaske wants to merge 5 commits into
gulp-community:masterfrom
magnuspaaske:addProcessName
Closed

Added process name/namespace option for compiling to client#157
magnuspaaske wants to merge 5 commits into
gulp-community:masterfrom
magnuspaaske:addProcessName

Conversation

@magnuspaaske
Copy link
Copy Markdown

Added 2 options for compiling to client JS with gulp-pug, namespace and processName. The namespace option creates a namespace so that all templates are compiled into the same namespace and can be easily found in the frontend code (as opposed to having all the templates being named 'template'). This change brings the processName option in line with grunt-contrib-pug (apart from a few details allowing this behaviour to only show up when the namespace option is set)

@jamen
Copy link
Copy Markdown
Contributor

jamen commented Jul 1, 2017

Hey @magnuspaaske, if you could provide some tests for this feature you are proposing, that would be nice.

(P.S. I don't use Pug much anymore, so let me know if you want access to maintain this better. 😄)

@magnuspaaske
Copy link
Copy Markdown
Author

Hi @jamen
I'll add some tests – I assume 2-3 tests should cover the functionality we're seeking here.
As for maintain access, let's discuss that in a separate thread :)
/Magnus

@phated
Copy link
Copy Markdown
Member

phated commented May 11, 2019

Closing this in favor of #199

@phated phated closed this May 11, 2019
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