-
Notifications
You must be signed in to change notification settings - Fork 118
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
Improve quality of existing code #6
Comments
Hey @Twiggecode I would like to work on this issue. May I? |
Anyone is free to contribute, there's no need to ask for permission or assign the issue to anyone, just have a look at the readme, create your algorithm or code changes and submit a pull request. |
Okay thanks for letting us know. I'll go ahead and read your documentation and see if there's anything I can do to help. |
@Twiggecode If we look at the How to contribute part in the ReadME.md half of it is basically explaining what this repository is all about and a very little part is about how to actually contribute to it. Since this is a beginner-friendly project can I add a couple of items like git commands and edit some parts of How to contribute? This would help them to create their first PR. |
I'd like to make a proposal. We could add a readme file to all of the folders that explains the algorithm with animation, or we could simply add a brief or a wiki link. |
I will try to go through already written algorithms and see if which one needs optimizations. |
@buggsaff Sorry I didn't receive a notification for this comment. There is a Wikipedia link in the readme that contains all of the integer sequences in this project. |
@Twiggecode , I have improved the quality of factorial series to o(logN). Please assign the issue and accept my merge request. |
Improve the quality of code that already exists within this project by either modifying the algorithms to become faster, or improve the coding standards by adding in more comments, spaces and by modifying variable names.
The text was updated successfully, but these errors were encountered: