Skip to content

correctly generate version export in main library - #3

Closed
johnyb wants to merge 1 commit into
openmason:masterfrom
johnyb:patch-1
Closed

correctly generate version export in main library#3
johnyb wants to merge 1 commit into
openmason:masterfrom
johnyb:patch-1

Conversation

@johnyb

@johnyb johnyb commented Oct 29, 2015

Copy link
Copy Markdown

since npm@3, node_modules directory might be flattened out. Since version is supposed to be the version of the "to" library, getVersion should explicitly provide the package.json of the project, instead of relying on handy to determine the correct one.

This fixes helpers/handlebars-helpers#199

since npm@3, node_modules directory might be flattened out. Since version is supposed to be the version of the "to" library, getVersion should explicitly provide the package.json of the project, instead of relying on handy to determine the correct one.

This fixes helpers/handlebars-helpers#199
@c-f-h

c-f-h commented Feb 11, 2016

Copy link
Copy Markdown

I've hit the same problem... unfortunately this package seems to be abandoned.

@c-f-h

c-f-h commented Mar 2, 2016

Copy link
Copy Markdown

Note: this PR has a bug itself, it tries to feed the path to package.json into getVersion(), which actually expects the package path. Even more, it creates the wrong path since it looks for a package.json in the same directory as to.js, whereas it should be one directory higher.

I submitted another PR which fixes this: #4

@johnyb

johnyb commented Mar 2, 2016

Copy link
Copy Markdown
Author

Oh, you are right … No need for this to stay open, then. Thanks for the information

@johnyb johnyb closed this Mar 2, 2016
@johnyb
johnyb deleted the patch-1 branch March 2, 2016 11:51
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.

handlebars does not support render.

2 participants