This issue was moved to a discussion.
You can continue the conversation there. Go to discussion →
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
Setting yarn
as the default
#102
Comments
Looks like it's not possible now but would be a nice addition so feel free to send a pull request 👾 One note though: What if make a |
Yeah, I thought about that. However, if we call:
it doesn't actually call yarn. I'd have to also put logic into If this is OK, I'll see if I can do that... |
This sounds like a bug or we don't really need this parameter there ;-) |
So, here's a couple of work-arounds: First
where Second
|
This might work as well as a workaround:
|
I'm using mrm task with programming api Currently To solve this issue, can we officially add an env variable, then in for example with programming apiprocess.env.MRM_INSTALL_USE_YARN = '1'
// ...then invoke mrm task with cliMRM_INSTALL_USE_YARN=1 mrm [task] I can send a PR if needed |
Just flying by this issue and would like to comment that we should not have yarn be the default, IMHO.... Defaulting to a tool that is not always available on a users system seems like a wrong direction. I would suggest that a smart resolution of package manager be used or stick with defaulting to For instance, I have adopted Just my $0.02 as a casual |
This issue was moved to a discussion.
You can continue the conversation there. Go to discussion →
OK, this may be another easy one I'm missing, but here goes: Assume you have a new project with this as your
~/.mrm/config.json
:If you run
mrm default
,prettier
begins installing usingnpm
.It seems like
getTaskOptions
inmrm/packages/mrm/src/index.js
is only using the default options if there are no parameters to the task.Or, should
mrm/packages/mrm-core/src/npm.js
re-read the globalconfig.json
to get theyarn
flag?The text was updated successfully, but these errors were encountered: