added windows os support, speed tested under 600ms!#56
added windows os support, speed tested under 600ms!#56TheTechsTech wants to merge 2 commits intospatie:masterfrom TheTechsTech:windows-patch
Conversation
To address some parts of issue #4 This is to make an record and to match https://github.com/uppes/parallel, before an big change in terminology used. For commits https://github.com/uppes/parallel/commits/master
|
For review, not sure windows support is really desired by your users. Anyway, I have different plans for the project. just showing it's possible without much overhead, and very close to the same speed results. |
|
Hey, pretty cool you got this working! Windows support is indeed not desired for this package, and I don't want to add stuff that we cannot justify maintain in the long run. Nevertheless thanks for sharing! I'll close the PR, but if someone really wants this package on Windows, they could look at this PR! |
|
Oh man, I would have LOVED seeing it work on Windows. |
|
The author introduced an excellent concept, but unfortunately made it very restricted from the start, and have recently even further stop testing on Windows. I have exported out various concepts and changes/rewrites thru different project names. |
…iginal author's `spatie/async` package. - Some these changes was part of previous PR fork spatie/async#56, others fix or address many issues raised in: spatie/async#168, spatie/async#147, spatie/async#162, spatie/async#164, spatie/async#168, spatie/async#160, spatie/async#153, spatie/async#152, spatie/async#157, spatie/async#134, spatie/async#76, spatie/async#107, spatie/async#98, spatie/async#4.
To address some parts of issue #4
This is to make an record and to match https://github.com/uppes/parallel, before an big change in terminology used.
For commits https://github.com/uppes/parallel/commits/master