-
-
Notifications
You must be signed in to change notification settings - Fork 15.3k
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
angular-language-server: collision with typescript #387400
Comments
Typescript isn't the kind of thing that goes in your system packages. Look into https://nix.dev/tutorials/first-steps/declarative-shell per project. |
Nix packages should generally not have the license present at root. It should go in a directory like |
@xdinterface Thanks for reporting this. I might need to adapt the file permissions of the node modules in the angular-language-server package. I'll have a look at this. |
While that is true, again, typescript doesn't go in one's config. That would avoid this becoming a blocking issue in the first place. |
I see your point about nix-shells being good practice, but I still think properly packaged Nix modules shouldn't collide regardless of how they're installed. |
Nixpkgs version
Describe the bug
I am trying to add the angular-language-server to my packages but it collides with my typescript package.
Steps to reproduce
Add typescript and angular-language-server to your packages and use nixos-rebuild switch to rebuild the system.
Expected behaviour
Both packages should be able to be installed alongside each other.
Screenshots
No response
Relevant log output
Additional context
No response
System metadata
"x86_64-linux"
Linux 6.13.5, NixOS, 24.11 (Vicuna), 24.11.20250302.1546c45
yes
yes
nix-env (Nix) 2.24.12
"nixos-24.05, nixos-unstable"
/nix/store/070ny7v4l1lbwx44ac3bsagzwx9mjdcy-source
Notify maintainers
@tricktron
Note for maintainers: Please tag this issue in your pull request description. (i.e.
Resolves #ISSUE
.)I assert that this issue is relevant for Nixpkgs
Is this issue important to you?
Add a 👍 reaction to issues you find important.
The text was updated successfully, but these errors were encountered: