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

Solve issue #244 #246

Closed
wants to merge 2 commits into from
Closed

Solve issue #244 #246

wants to merge 2 commits into from

Conversation

jfoclpf
Copy link

@jfoclpf jfoclpf commented Feb 24, 2019

Solve issue #244. When viewsPath is truthy, the partialsDir which have the directories defined by the user on the constructor, are overwritten by as single String path.join(viewsPath, 'partials/'). I'm playing safe here, I assume partialsDir is internally always dealt as an Array and therefore I convert it to an Array, pushing when appropriate the default path.join(viewsPath, 'partials/'), but keeping the previously defined partialDirs

Solve issue #244. When `viewsPath` is truthy, the `partialsDir` which have the directories defined by the user on the constructor, are overwritten by as single String `path.join(viewsPath, 'partials/')`. I'm playing safe here, I assume `partialsDir` is internally always dealt as an Array and therefore I convert it to an Array, pushing when appropriate the default `path.join(viewsPath, 'partials/')`, but keeping the previously defined `partialDirs`
@sahat
Copy link
Collaborator

sahat commented Feb 24, 2019

Thanks for submitting a PR. I went ahead and merged #245 reviewing both PRs and following the discussion on #244. The newest version v3.0.2 should now be published on NPM.

@sahat sahat closed this Feb 24, 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.

None yet

2 participants