-
-
Notifications
You must be signed in to change notification settings - Fork 323
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
import-js compliant #1053
Comments
@zirho Hi. I'm not aware of this |
Hey Victor, Thanks! |
@zirho Yes, the dependencies need to be populated as well, but I guess not all of the dependencies, we certainly don't want to declare |
@zirho ... though if we have |
@Vlasenko Although I don't completely understand lerna architecture, just skimmed through the doc briefly, I see what you are saying. Anyway, it seems importjs tries to parse the file and add import code when I add sibling packages in There are certainly few more hoops required to get this working. Let me dig more into it. |
Kutos to Lerna!
Hey guys, thank you for the great work.
Just wanted to fire up a conversation about recent Lerna migration.
I believe that it will give us a lot of advantages on maintainability in a long run.
Here is a thing I believe this project can do better.
For me, It became a second nature to use import-js to import other files' exports automatically.
Recent Lerna integration totally dismissed the advantage of it and affecting productivity due to the missing property 'main' in each of package.json.
At the same time, I am not 100% sure that I set my dev environment correctly so that auto import to work.
For me, I have to calculate how many folders I should hop through manually to import the file correctly.
Which is totally feasible compare to the advantage this project gives but I feel like it can be improved.
Do you guys have plan to battle that? or can you explain the way you guys auto-import exports if yours work?
Thanks!
related discussion
Galooshi/import-js#504
The text was updated successfully, but these errors were encountered: