aurelia / ux

A user experience framework with higher-level capabilities, designed to bring simplicity and elegance to building cross-device, rich experiences.
MIT License
368 stars 55 forks source link

Multiple versions of @aurelia-ux/core found: #201

Closed stuartbale closed 4 years ago

stuartbale commented 5 years ago

I'm submitting a bug report

Please tell us about your environment:

Current behavior: I have followed the instructions in a new Aurelia project, to add aurelia-ux. When building the project, there are various warnings: WARNING in @aurelia-ux/core Multiple versions of @aurelia-ux/core found: 0.16.0 ./~/@aurelia-ux\core 0.4.0 ./~/@aurelia-ux\slider/~/@aurelia-ux\core

WARNING in aurelia-dependency-injection Multiple versions of aurelia-dependency-injection found: 1.3.1 ./~/@aurelia-ux\slider/~/@aurelia-ux\core/~/aurelia-dependency-injection 1.4.2 ./~/aurelia-dependency-injection

WARNING in aurelia-metadata Multiple versions of aurelia-metadata found: 1.0.3 ./~/@aurelia-ux\slider/~/@aurelia-ux\core/~/aurelia-metadata 1.0.6 ./~/aurelia-metadata

WARNING in aurelia-pal Multiple versions of aurelia-pal found: 1.3.0 ./~/@aurelia-ux\slider/~/@aurelia-ux\core/~/aurelia-pal 1.8.2 ./~/aurelia-pal

It appears to me that the slider control has invalid and/or unneeded references

Expected/desired behavior: When adding aurelia UX to a new Aurelia project, it should not have warnings when building.

stuartbale commented 5 years ago

OK - I think I can see the issue. It appears that the slider component is including it's own node_modules folder as per the attached image. I'm hoping this is something that is easily fixed.

Manually deleting the node_modules removes the warning, and the app runs correctly. Screen Shot 2019-06-21 at 1 35 52 PM

bigopon commented 4 years ago

This should be solved by recent work from @MaximBalaganskiy that put @aurelia-ux/core as peer dependency. Thanks for creating this issue.