José M. Pérez

Results 43 comments of José M. Pérez

Hi @Trevahok, I had a look at the files, and you are definitely right. The are not being converted, but it's also true that some of them are not part...

Hi @Jakobud! The auto generated documentation is missing some information that does exist in the README. Thus, it is better to check the README file for the time being. For...

You are right, @solmsted. As far as I know, `AbortSignal` and `AbortController` are not supported in Node.JS. One reason is that `DOMException` is not defined in Node.JS. [According to the...

Thanks for your PR @ilkerc! Since the library is used both in the browser and in node, using `events` means that `events` would need to be included in the client-side...

@ilkerc I'm doing some cleaning and tidying up my repos. I wanted to address this issue. One of the things I noticed is that the requirement of the 'events' library...

At the moment if you pass the signal to a single promise it will cancel that promise. The library doesn't take into account the case in which you are using...

@parsasaeedi Yes, this looks definitely like a problem as your use case should be supported. This is a bug that should be addressed.

Could you share a link to the playlist that gets stuck? Also, if it does delete duplicates, does it mean it eventually doesn’t find duplicates if you run it several...

Just for completeness, there are corner cases that are difficult to fix because they are hard to reproduce.

Wow, thank you so much for the detailed report, @shelaffs! I tried finding out where the problem was by checking at Sentry, the error reporting tool I use, but I...