search

clarkie / dynogels

DynamoDB data mapper for node.js. Originally forked from https://github.com/ryanfitz/vogels
Other
491 stars 110 forks source link

Project is dead? #180

Closed rchl closed 5 years ago

rchl commented 5 years ago

Given that autor didn't respond for over a year, I think the only sensible solution is to fork the project and integrate some much needed PRs.

I can't volunteer much time to it myself but hopefully someone will.

I propose new name - dynogelsNext

cdhowie commented 5 years ago

I have commit access but I don't believe I have the required keys to publish new versions to npm.

thomasruiz commented 5 years ago

@clarkie ?

diosney commented 5 years ago

Bump.

It is a shame that this project goes to waste, by only reading the README it can be seen that is huge and very useful, but it is too dangerous to use a stalled project :/

cdhowie commented 5 years ago

@diosney It could be forked... dynogels is a fork of vogels, which died some time ago when the maintainer disappeared.

But I would prefer if @clarkie could come back to at least hand over the reins, instead of having to change the name of the project a second time. 😞

diosney commented 5 years ago

@clarkie Agree completely.

snamoah commented 5 years ago

created a new project based on this and dynogels-promisified. you can check it out here https://www.npmjs.com/package/dynogels-promise. I updated the dependencies and made some few changes but you can replace it with either packges, and it should work perfectly

clarkie commented 5 years ago

Yo, apologies for going a bit off the grid. I've completely stopped using dynamodb (for now).

I'm happy for this to be moved over to a group/org type thing. What needs to happen to get this set up? I've got some time over the next few weeks where I should be able to hand this over to whoever wants to carry on maintaining it.

I think there's two bits to do, github and npm. Anything else?

cdhowie commented 5 years ago

I think there's two bits to do, github and npm. Anything else?

Probably the CI stuff as well. If the integration tests hit a real DynamoDB then we have to figure out what to do with that AWS account.

clarkie commented 5 years ago

If I remember correctly the CI spins up a "local" dynamodb to run against

cdhowie commented 5 years ago

Cool, so in that case just access to the CI backend is an additional piece that would have to transition.

MaximusHaximus commented 5 years ago

I'd like to volunteer to help get V10 landed and help out with getting things converted to an org if needed. Have been using dynogels for 3 years and would like the chance to give back :) @cdhowie @clarkie

Tombmyst commented 5 years ago

Any news on project update? I'm currently working on updating it and re-writing it in Typescript

clarkie commented 5 years ago

Sorry! What's the best structure for the new group? Is it a github org and an npm org? I've never set up an npm org before.

Would the group be just dynogels/dynogels on github and @dynogels/dynogels on npm?

snamoah commented 5 years ago

Creating an organisation on npm is quite straightforward... the organisation structure on github is fine.. we can still keep dynogels name the same on npm imo

jvgeee commented 5 years ago

Any movement on this? @clarkie

ErikBoesen commented 5 years ago

https://docs.npmjs.com/creating-an-org This is how you make an npm org. There are some vulnerabilities in this package's dependencies so it would be nice to get new maintainers access to upload there.

nolde commented 5 years ago

We really need to get this going. I am willing to help.

I am already thinking of creating yet another fork to cater to my business needs. I would rather do a PR for dynogels.

clarkie commented 5 years ago

Thanks @ErikBoesen I've created the npm org now I just need to invite @cdhowie in. Can you ping me your email or npm username.

clarkie commented 5 years ago

This will still move the npm dependency though right? So we'll go from yarn add dynogels to yarn add @dynogels/core or something similar?

clarkie commented 5 years ago

Oh hang on, we don't need to do all this. I can just add maintainers to the existing npm package.

I still need @cdhowie's (and anyone else who y'all think should have publish rights) npm usernames

ErikBoesen commented 5 years ago

I agree that it would be preferable to just add maintainers.

cdhowie commented 5 years ago

@clarkie My username is the same over there. :)

nolde commented 5 years ago

Any news?

clarkie commented 5 years ago

I've added @zolaemil and @cdhowie as maintainers on npm

zolaemil commented 5 years ago

Thanks @clarkie!

zolaemil commented 5 years ago

I've now merged https://github.com/clarkie/dynogels/pull/155 and https://github.com/clarkie/dynogels/pull/179 and going to publish a release to npm with these soon.

zolaemil commented 5 years ago

I realise that master has some breaking changes since v9.0.0, so in order to move forward quickly with addressing vulnerabilities, I've decided to port these dependency updates back to v9 and make a release from there.

https://github.com/clarkie/dynogels/releases/tag/v9.1.0