| 31 Jul 2023 |
 Lily Foster | * And I was going to say you could maybe get away with packaging the tarball for pyright from the npm registry, but it looks like that doesn't contain a lockfile | 21:28:03 |
| Lily Foster | In reply to @robert:funklause.de
Then I guess there are more important things to work on If you want to work on that, feel free to, but it'll be a while before we get there. We really should come up with at least some workarounds though for packaging lerna apps since they keep popping up (even though npm has native monorepo support with workspaces now which we do support since it didn't actually take that much extra effort) | 21:29:30 |
| Lily Foster | (I'm not even sure what's the best way to handle lerna tbh. I've never used it and it's been rather annoying when toying with trying to build lerna repos with npm) | 21:30:58 |
| Lily Foster | * (I'm not even sure what's the best way to handle lerna tbh. I've never used it and it's been rather annoying when toying with trying to build lerna repos with npm alone) | 21:31:07 |
| Lily Foster | (if you really want though, you could use the npm registry tarball as src and then copy in a vendored lockfile in postPatch. But I really don't want to vendor any more lockfiles than we have to in nixpkgs.....) | 21:31:47 |
| Lily Foster | (especially with how big npm ones get) | 21:32:04 |
 gdamjan | In reply to @gdamjan:spodeli.org
why didn't I see this though?! I did run the tests too 😱 | 21:54:04 |
| 1 Aug 2023 |
| gdamjan | how exactly did you get this? just running the CLI doesn't error out for me | 07:59:03 |
 hexa | so did I | 10:54:28 |
| hexa | ofborg also ran them and they were green before merge | 10:54:40 |
| gdamjan | what's the game plan now? add the package.json file back, and perhaps add strictDeps=true? | 11:11:09 |
| hexa | yes | 11:38:38 |
| gdamjan | https://github.com/NixOS/nixpkgs/pull/246554 | 11:58:57 |
 dotlambda | Do we want to support npm-shrinkwrap.json in buildNpmPackage? | 16:01:49 |
| Lily Foster | In reply to @robert:funklause.de
Do we want to support npm-shrinkwrap.json in buildNpmPackage? If someone wants to write it. Right now it's not on our radar though | 16:03:53 |
