-
Notifications
You must be signed in to change notification settings - Fork 6
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
Adopt rethinkdbdash definitions #11
Comments
Nice work! You're welcome to move the repo to @types. Let me know add you so you can move it 😄 |
@blakeembrey ok sounds good, yea I'm happy to move it over. |
@blakeembrey I have transferred the repo: https://github.com/types/npm-rethinkdbdash Can you walk me through the next steps? I don't think I have the ability to enable travis or greenkeeper per the guidelines. I imagine we'll also need a PR to the registry repo as well? |
I'll rename the repo to Yes, a PR to registry. But let's get the travis up first. |
Yep, that would be great. @unional We can definitely start doing that overall. This is related to a greater discussion for all @types though to align with NPM |
@blakeembrey I think I have finally accomplished what you were getting at in our previous discussion (#8). I have the repo up here: https://github.com/marshall007/npm-rethinkdbdash.
Apologies again for copying the definitions into that DefinitelyTyped PR, I just didn't understand enough about how all this works at the time to appreciate what the goals are here.
@blakeembrey @niieani I hope you guys will consider migrating my repo to the @types org and publish them to NPM.
Cheers!
The text was updated successfully, but these errors were encountered: