Add support for yarn linked generators #86
Conversation
lib/resolver.js
Outdated
| // To ensure `yarn global add yo` and `yarn link generator-i-made` work as | ||
| // expected, add the `link` folder to path if we appear to be installed in | ||
| // yarn's global tree. | ||
| if (path.basename(path.join(__dirname, '../../../../')) === 'yarn') { |
There was a problem hiding this comment.
Any reason/tradeoff to not add paths.push('~/.config/yarn/link/') directly? I feel that'd be simpler and less tricky
There was a problem hiding this comment.
Because I wrote this immediately after I woke up and didn't realize there was an absolute path. Will send fix.
There was a problem hiding this comment.
Alright, I mean this could've been for a reason - like maybe yarn wouldn't always use the same config folder or something. But alright, I'm happy with the simpler solution :)
There was a problem hiding this comment.
I honestly don't know much about yarn installations on other machines.
The original solution would be safer if it moved around, but I don't know if it does.
|
Thanks for your help! I'll publish a new release by the evening in my TZ. |
|
And it's released as 2.0.2 - thanks again for the PR! |
Closes yeoman/generator#1032