-
Notifications
You must be signed in to change notification settings - Fork 14
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
Set up monorepo with lerna #92
Conversation
Got this set up, but how does it work in practice? How do I require it in a project? Lerna docs seem thin on this. |
This doesn't affect the way you use the library or its plugins, since right now there is only the one package called The main repo postinstall script will go install dependencies in all the subprojects. Being new to this, I'm not sure what else there is to document… |
I mean that when I change my dependencies to include
then run my tests that begin
I get |
I gotcha – you're wanting to use it through vcs. I'm not totally sure, to be honest, that's a good question. Out of curiosity, are vcs links like that something you typically commit, or just use locally? |
Only in instances like this, to test a branch. Do you expect this to work
differently when npm published?
…On 7 Dec 2017 17:56, "Paul Melnikow" ***@***.***> wrote:
I gotcha – you're wanting to use it through vcs. I'm not totally sure, to
be honest, that's a good question.
Out of curiosity, are vcs links like that something you typically commit,
or just use locally?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#92 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ACBN2xyV_pATesCmQEMG-_cXx-5xpQplks5s-CbBgaJpZM4QMEHe>
.
|
Yea, when it's published it should work just like before. Have you tried using |
If you're able to get the local module working, feel free to push some doc to this branch, or let me know and I can add it. I'm eager to get this in! |
Every method still ends up with that same folder structure of icedfrisby-monorepo\packages\icedfrisby, and fails the require('icedfrisby') npm link docs even say
https://docs.npmjs.com/cli/link Maybe we should npm publish as another tag and try from there? |
Did you try running |
If I link to the package, the diff is pretty a reliable indicator that this will work. Is there a way we can test that the lerna config is gonna work? Gut feel to progress is that we need to be confident in 2.0a1 and merge this in and make it 2.0a2. I'll start taking a look at a1 now. |
I've already spotted one thing though - I can't use eslint in the repo as it stands. The monorepo package doesn't have the lint command, and the subpackage doesn't have a reference to eslint. Not sure what the preference for this is? |
All of the 2.0a1 unit tests pass, the docs look good, and my work repo of ~250 checks (inc ones with plugins) all run fine using 1.5 and 2.0a1. |
I've been switching from Greenkeeper to Dependabot on other projects and have found it much more reliable, especially for dealing with lockfiles. Dependabot now supports Lerna: |
Gonna close this for now. |
I took a stab at this. Wasn't too bad. Let me know if you have questions!
Todo