Skip to content
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

new maintainer #21

Open
legokichi opened this issue Jan 21, 2019 · 12 comments
Open

new maintainer #21

legokichi opened this issue Jan 21, 2019 · 12 comments

Comments

@legokichi
Copy link
Owner

i dont have enough time to maintain this repository
anybody help?

@guest271314
Copy link

Willing to help where able.

@marob
Copy link

marob commented Feb 22, 2022

@guest271314 I see you're pretty active on this unmaintained package. Would you consider becoming the new maintainer?

You could ask @legokichi some contributor role on his repository and becoming owner of the package on npmjs repository, or decide to publish your fork as a new package that would become the de facto maintained version of this package.

I'm particularly interested by having a JavaScript module version of this package in order to be able to use it in slidev: slidevjs/slidev#491

@guest271314
Copy link

I am not familiar with TypeScript. I am willing to help where I can.

You should just be able to use the Ecmascript module version of ts-ebml. I also created an exportable version of lamejs for https://github.com/guest271314/captureSystemAudio/tree/master/native_messaging/capture_system_audio.

@marob
Copy link

marob commented Feb 22, 2022

The goal is indeed to be able to use the Ecmascript module version of ts-ebml in slidev (slidevjs/slidev#491).
But in order to be able to do that, we need to have this Ecmascript module version available somewhere in an npm package.
If I'm not mistaking, Ecmascript module version is not available in ts-ebml npm package (as it's only in your fork sources), nor in any other npm package.

I did find the Ecmascript module version:

@guest271314
Copy link

I have only worked on 1 npm package. What I suggest is just using the minimized Ecmascript module version.

we need to have this Ecmascript module version available somewhere in an npm package

Why?

All of the necessay code is in the single script.

          const {
            Decoder,
            Encoder,
            tools,
            Reader,
            injectMetadata,
          } = await import(`${this.src.origin}/ts-ebml.min.js`);

@marob
Copy link

marob commented Feb 23, 2022

we need to have this Ecmascript module version available somewhere in an npm package

Why?

Because using dependency management is good practice.
If we depend on some JavaScript available at some URL at runtime, it can fail if it's unavailable, if the user is not connected, ...
If we duplicate the JavaScript code in slidev, we'll never benefit from any update/fix of the original ts-ebml.

That's why we need the Ecmascript module version in a published npm package on the npmjs registry.

@guest271314
Copy link

@marob

You could ask @legokichi some contributor role on his repository and becoming owner of the package on npmjs repository, or decide to publish your fork as a new package that would become the de facto maintained version of this package.

I am willing to help where able, particularly with testing. I have no experience with TypeScript. Are you committed to maintain the TypeScript in this repository?

@davedoesdev the owner of https://github.com/davedoesdev/webm-muxer.js which also uses https://github.com/muaz-khan/RecordRTC/blob/master/libs/EBML.js could also be a potential candidate for maintainer.

@marob
Copy link

marob commented Feb 24, 2022

@guest271314 I just need this package as a dependency and have no expertise in video manipulation. Also, I haven't read the code of ts-ebml. I don't see myself maintaining this package.

TypeScript is everywhere today in JS world. You should give it a try 😉

So, we still don't have a solution for now...

Maybe @muaz-khan could also be a candidate as maintainer of ts-ebml as it's used in https://github.com/muaz-khan/RecordRTC?
In addition, RecordRTC is the package used in slidev and is the one that should depend on ts-ebml in my opinion.

@guest271314
Copy link

@marob Can't you just merge the PR I filed into your fork and publish your fork on npm?

@marob
Copy link

marob commented Feb 24, 2022

@guest271314 I could, but then we would have 2 unmaintained packages.

@davedoesdev
Copy link
Contributor

I can help out, can't promise super quick turnaround though.

@marob
Copy link

marob commented Sep 12, 2022

Hi @davedoesdev. How can you help? Are you ready to maintain a fork of this repository?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants