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

Looking for maintainers #66

Open
nothingisdead opened this issue Jan 27, 2020 · 15 comments
Open

Looking for maintainers #66

nothingisdead opened this issue Jan 27, 2020 · 15 comments

Comments

@nothingisdead
Copy link
Owner

Hi,

I haven't really had time to maintain this library for a while now, and I don't use Kraken anymore so it's difficult for me to test pull requests. I know that it's still used in a lot of projects, so if someone would like to volunteer to maintain it, please let me know!

@chuckamus-prime
Copy link

how much do you pay? JK. I use this library quite a bit. for your consideration, I will volunteer to help maintain it. if you want to chat/cam some time about it, id be happy to meet you and understand your rules/guidelines for doing so.

@nothingisdead
Copy link
Owner Author

Thanks!

I just merged a few of the pull requests that didn't look like they required testing. To start with, I would love it if you could look at any of the other open issues or pull requests, specifically:

For #63 I'd rather clone the params object to avoid mutating the params object that was passed in, as suggested:

Better solution would be to not attach the nonce property to the params object passed from userland.

#57 doesn't seem like it would cause issues, but I'm hesitant to merge it without testing.

#54 conflicts now, and I'd rather it be split into separate pull requests, but it would be great to add unit testing.

If you'd like to work on any of those (or others), can you submit a separate pull request?

@nothingisdead
Copy link
Owner Author

nothingisdead commented May 16, 2020

#49 would be another good one to take a look at. I don't have a strong opinion on got vs needle, but if I were to change it, I'd probably switch to node-fetch since it follows a standard (and it has no dependencies).

@chuckamus-prime
Copy link

Thanks, I will start looking into them, and yes, if i work on them i will supply separate PR.

@ctwhome
Copy link

ctwhome commented May 19, 2020

hi @nothingisdead , what do you use now instead of kraken?

@Amantel
Copy link

Amantel commented Oct 26, 2020

Hi, guys!
Who is the maintener now?

@elio-maggini
Copy link

@nothingisdead Hello. I am just getting going with this project but reaching out in case it is still possible to get involved as a contributor. Could at least get started with #54 as unit tests are a great way to start learning a pre-existing code base.

@lnxslack
Copy link

@nothingisdead i would like to help to maintain it

@elio-maggini
Copy link

@lnxslack I have not yet received a reply either. But perhaps you would be interested in involvement in this version....

https://github.com/elio-maggini/kraken-nest

@lnxslack
Copy link

@elio-maggini i'm starting to nodejs, i will need help to follow you on this. May i share some thing i did using kraken-api

@elio-maggini
Copy link

Sure thing. Send the github link :)

@KeesCBakker
Copy link

Any updates on maintenance? Looks like the hardest part is the no test suite-part :-)

@dscotese
Copy link

dscotese commented Dec 5, 2021

I just made an npm account because I want to publish https://github.com/dscotese/kraken-grid. Anyone with an account there can fork this repo and create a new package on npm but it would be cleaner to follow the process at https://docs.npmjs.com/transferring-a-package-from-a-user-account-to-another-user-account once someone steps up who is willing to be the maintainer. I'm close to doing that, but I worry that there is information I won't be able to get fast enough because I don't know any npm package maintainers. If Robert figures it's easy enough, he can follow those instructions with my username (same as here on Github) and I will do my best.

@lnxslack
Copy link

lnxslack commented Feb 8, 2022 via email

@dscotese
Copy link

dscotese commented Feb 9, 2022

I copied this repo to https://github.com/dscotese/kraka-djs which I am maintaining on my own. You'll see two fixes (#77 and #73) in there. It's also on npmjs.com

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

8 participants