-
Notifications
You must be signed in to change notification settings - Fork 30
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
Open Source Contributions #43
Comments
Hello @abhisheksoni27 Yes, would love that. If enough members line up for this, can create guidelines on contributions and maintain an open issues list. Would like to see some response here before moving in that direction. There is one caveat, which is we will have to keep issue resolve time on the lower side, as the purpose of the library is to ensure the users do not face any issues while using it for their trading setup. |
Thanks, @dhan-oss! Can we keep the issue in opened state otherwise highly unlikely we would get any response here. :) |
@dhan-oss What is your lead time to merge a PR? Mine is in waiting state still. Any feedback is much appreciated. |
Hey @karthiks It usually takes 2-3 days for us to approve and merge any PR. Since PRs created by you require testing all the functions, it is taking a bit longer. Will be merging your PR soon. |
Hi, Team
First of all, thank you for this great library.
As a developer myself, I wanted to know if there are set of good-first-issues that you guys are waiting to develop here?
Or
Maybe there are things on the roadmap that the open source community can help you guys out in?
I would love to contribute to this and wanted to reach out.
Looking forward to your response.
The text was updated successfully, but these errors were encountered: