Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • 11:19
    zzuligy commented #3952
  • 08:49
    zkochan commented #5408
  • 04:35
    yn-tadpole edited #5424
  • 04:34
    yn-tadpole opened #5424
  • 04:34
    yn-tadpole labeled #5424
  • 04:26
    yn-tadpole labeled #5423
  • 04:26
    yn-tadpole opened #5423
  • 04:10
    yn-tadpole edited #5422
  • 04:00
    yn-tadpole edited #5422
  • 03:54
    yn-tadpole labeled #5422
  • 03:54
    yn-tadpole opened #5422
  • 03:28
    Shinyaigeek commented #5408
  • Sep 28 20:17
    zkochan synchronize #5408
  • Sep 28 20:17
    adam-coster labeled #5420
  • Sep 28 20:17
    adam-coster opened #5420
  • Sep 28 18:54
    d4mr commented #5351
  • Sep 28 15:17
    Shinyaigeek synchronize #5408
  • Sep 28 14:58
    Shinyaigeek synchronize #5408
  • Sep 28 14:24
    Shinyaigeek synchronize #5408
  • Sep 28 13:41
    raineorshine edited #5419
Zoltan Kochan
@zkochan
@JR-Utily a dependency with no spec? If it works with npm then we'll fix it, please create an issue for it
Nik
@nikdojo
Sure, I'm just testing the whole day today with all those fixes for Ember resolver. It works better now, but new issues appear, including this one.
JR Utily
@JR-Utily
this is while including https://github.com/aam229/local-dependencies in my deps
Zoltan Kochan
@zkochan
so you say the "inquirer-path": "*", part fails? That would be weird but I'll recheck tonight. The fix will be easy if that is the case
JR Utily
@JR-Utily
that's it pnpm/pnpm#865
thanks a lot
JR Utily
@JR-Utily
@zkochan thanks for the fix :)
Zoltan Kochan
@zkochan
you're welcome
Wei Wang
@onlywei
Hi, what happened to the README? https://www.npmjs.com/package/pnpm
Zoltan Kochan
@zkochan
I don't know, I published it the same way I always did
the readme is in the registry. Maybe the npm website has issues
Wei Wang
@onlywei
Spent all day making this reproducible error case for pnpm: pnpm/pnpm#867
Wei Wang
@onlywei
how does pnpm determine what URL to fetch from?
Sometimes it tries to hit:
://npm-registry.domain/p/pnpm/_attachments/pnpm-1.10.1.tgz (correct)
And sometimes it tries to hit:
://npm-registry.domain/pnpm/-/pnpm-1.10.1.tgz(causes 406)
Zoltan Kochan
@zkochan
sometimes from the shrinkwrap file and if there is no such then from the dist field of the metadata.
Zoltan Kochan
@zkochan
@onlywei the fix is ready for CR pnpm/supi#6
Wei Wang
@onlywei
@zkochan thanks! What's the best way for me to test this before you cut a release?
Zoltan Kochan
@zkochan
you can clone pnpm from master and supi from the fix/npm-enterprise branch. Link supi to pnpm, run npm run tsc in both projects
and then you can use pnpm from the cloned repo
Joe Schmitt
@josephschmitt
@zkochan gonna give this a go
thanks for looking into it and fixing so quickly. this one was killing us all week until we were finally able to make a narrow test case
Zoltan Kochan
@zkochan
np. It is a pleasure to fix something when you already know what is the issue
you did all the hard work
Joe Schmitt
@josephschmitt
@zkochan hmm, not entirely clear on which supi link function i should be running. is it just supi.link()? and what are the params?
Zoltan Kochan
@zkochan
I mean link the cloned supi repo to pnpm
pnpm link ../supi (from the cloned pnpm repo)
Joe Schmitt
@josephschmitt
ooooh, gotcha
Joe Schmitt
@josephschmitt
@zkochan getting this error: ERROR ENOENT: no such file or directory, open '/usr/pnpm-global/node_modules/supi/package.json'
does this not work with locally installed pnpm?
i ran ./node_modules/.bin/pnpm link supi/
Zoltan Kochan
@zkochan
better try with the globally installed one
and using a relative path
Zoltan Kochan
@zkochan
@josephschmitt I can just publish it
all tests are passing so it should work
Joe Schmitt
@josephschmitt
@zkochan ok, sounds good. we can install and test within the hour
Zoltan Kochan
@zkochan
ok
Wei Wang
@onlywei
testing now
Wei Wang
@onlywei
seems promising so far
had to regenerate my shrinkwrap.yaml files, which is to be expected
Wei Wang
@onlywei
great success!
Zoltan Kochan
@zkochan
:tada:
Zoltan Kochan
@zkochan

I am rewriting parts of pnpm using reactive programming. It looks a lot better and I think it will help to write more complex/performant algorithms.
However, I am just learning reactive programming, so I'd appreciate if someone more experienced could do code reviews

has anyone experience with reactive programming?

Zoltan Kochan
@zkochan
In case someone is interested in the reactive rewrite: pnpm/supi#8
Vaughan Rouesnel
@vjpr
@zkochan can you point me in the right direction for pnpm/pnpm#600
Zoltan Kochan
@zkochan
I am on vacation, so unlikely will I contribute this week but I will try to answer shortly for any questions.
I'll read the issue and try to give some clues
Vaughan Rouesnel
@vjpr
thanks, just the old Z_BUF_ERROR error...
Zoltan Kochan
@zkochan
If this is concurrency issue maybe you can try to throttle unpack stream or whatnot
Vaughan Rouesnel
@vjpr
it seems to be happening predictibly for me at the same point. i will see what tar is causing it today.
so i dont think its concurrency….maybe an unpack-stream edge case
Daniel Bayley
@danielbayley
@zkochan What are your thoughts on implementing features that the npm core team refuse to consider? Requests such as being able to use other formats like YAML instead of package.json, and more powerful lifecycle scripts keep cropping up for example. I am working on a bunch of ideas to solve these frequent annoyances for myself in the form of more npm modules (and others have attempted with modules like nps and npm-run-all), but ultimately there would be a lot less need for hacky/workarounds if such features were supported in [p]npm core. So I'm just wondering whether your opinion is generally "pnpm is solely focussed on speed/disk space efficiency over regular npm", or more "open to other improvements over npm, which can help pnpm gain more traction"… what are your thoughts?
Zoltan Kochan
@zkochan
@danielbayley I am not sure about adding it to core yet but I am fine creating a "hook system" that will allow to do things like you described. See: pnpm/pnpm#861
Daniel Bayley
@danielbayley
@zkochan Interesting… I'll check it out thanks!
Zoltan Kochan
@zkochan
Just some status updates. I am on vacation this week.
Probably I won't add new features to pnpm v1. I might fix some bugs till v2 is not ready.
For the v2 beta I have to finish the "reactive" rewrite of pnpm, see PR: pnpm/supi#8
Vaughan Rouesnel
@vjpr
@zkochan could you explain package-store/lib/network/got.js#download?