-
Notifications
You must be signed in to change notification settings - Fork 12k
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
[beta.31] npm WARN at installation in global #4575
Comments
Again, as a workaround, installing all these modules globally after the update of @angular/cli makes everything work correctly for me:
|
We still need these dependencies as they are peerDeps of @ngtools/webpack. Partially revert angular#4473 Fix angular#4575
We still need these dependencies as they are peerDeps of @ngtools/webpack. Partially revert angular#4473 Fix angular#4575
We still need these dependencies as they are peerDeps of @ngtools/webpack. Partially revert angular#4473 Fix angular#4575
We still need these dependencies as they are peerDeps of @ngtools/webpack. Partially revert angular#4473 Fix angular#4575
We still need these dependencies as they are peerDeps of @ngtools/webpack. Partially revert angular#4473 Fix angular#4575
workaround suggested by @AlainD- indeed works.. However, that may not be acceptable in long run.. I could see that @filipesilva had committed a fix yesterday. However, could anyone please let me know in which release it will be made available? |
OS: Linux (Ubuntu 14.04 LTS) The application was initially created with angular-cli 1.0.0-beta.26 Versions
Steps to reproduce First, I removed previous angular-cli and cleared the cache,
Then, I applied @AlainD- workaround,
No visible failure (traces) It is tricky but compiler detects changes & my browser gets updated as if all was working perfectly! But, when I tried to change some code, it does not reflect it! Take the following scenario, Before applying @AlainD- workaround, my callback was printing titi on a click event.
What am I missing ? Any help appreciated :) |
My guess is that since you did not stop ng serve during the update, your ng command was the one from [email protected], serving the code compiled with the old angular compiler. I supposed that after having stopped ng serve, the removal of the remaining artefacts/cache/memory of the old angular-cli has been flushed. And when starting the command But it is not clear if you (still) have an issue after restarting ng serve. Probably you could clarify that last point. |
Sorry, I was not clear enough :) Initially, I bumped into this issue,
I finally got to upgrade and apply your solution (@AlainD-), hoping it fixes my initial issue!
No I do NOT have any changes reflected
As described in my previous message, when i change my code, it is well detected (thanks to your workaround) and my browser gets refreshed, but the code running does not reflect the changes! Hoping being more clear, Thanks for your help :) |
@alaind : Because of limited cli experience (and possibly low intelligence 😀 ) I am not sure how to interpret your workaround. It consists of three lines, all beginning with "npm install -g", of which I had already run the first one, which is what gave me the warnings of unmet dependencies that led me here eventually. Therefore, I concluded I should begin with the second line of your workaround, and I did, and it complained about the following unmet peer dependencies:
Next, following my interpretation of your workaround, I ran the third line, which addressed two of the unmet dependencies from above and which led to no complaints at all. Now I am wondering if this is really it, and I would guess it isn't since, for example, I have done nothing (that I know of) about some of the aforementioned unmet peer dependencies. Plz help! I have an exam tomorrow and cannot aquire ng-skills without this. /Simon Oh, and my OS is Windows 10 Education x64 and I am running commands from an elevated PowerShell. |
@nam4dev I'm not a developper of angular/cli nor a contributor to this repo. Just a guy trying to learn, as much as you, this new stuffs. I just figured that providing valuable details would help the contributors of angular cli to help you on your issue. @simonefternamn That's exactly how i processed my problems when I got them. I just read the warning messages, and decided to install the missing dependencies manually afterward to try to fix them, and that worked for me. |
It seems like I had to install all those packages with one single npm install -g-command in order to not get any warnings. I don't know how to check what is installed, however, and what I needed the installation for still doesn't work, unfortunately. Thanks anyway, @AlainD- |
@AlainD- Of course, I understand well your point and I thank you for your time and help! Anyway, I finally got it to work properly ;) Using Jet brains IDE, I already fixed inotify with suggested value 524288 as described here. For those who may encounter the same issue as me, below what fixed it:
Restart your IDE |
@nam4dev I read here #2389 (comment) that modifying those settings are not advised, and could cause memory issues instead. |
@simonefternamn normally you should have at the end the npm install a summary of all the detected warnings and errors found during the installation. So if you did not get anything, it's fine. For info, you could run a npm list -g to list all the packages installed globally. or npm list -g a-package-name to check the installation (globally) of the specific package a-package-name. Same goes for packages in a local project by removing the -g option in the command line. |
We still need these dependencies as they are peerDeps of @ngtools/webpack. Partially revert angular#4473 Fix angular#4575
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
When I run
npm -g i @angular/cli
, warnings are shown as the below:OS?
Versions.
@angular/cli: 1.0.0-beta.31
node: 6.9.5
os: darwin x64
Repro steps.
The log given by the failure.
Mention any other details that might be useful.
The text was updated successfully, but these errors were encountered: