-
Notifications
You must be signed in to change notification settings - Fork 5
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
Refactor index.js and clean-up code #6
Comments
Hey, Can I take this up? |
@MKPATH2015 Sure, thanks! |
I'm interested, too. |
@thiagojmlins and @MKPATH2015 you could try collaborating or @thiagojmlins you can check out the other issues in the repo as well. |
Awesome 👍 |
@thiagojmlins, @MKPATH2015 has added his version of the refactor(merged). If you can further make some improvements, you can let me know, if not, I will close the issue. |
Hm, great. Maybe I could migrate this to TypeScript :) |
Sure, go ahead
Regards,
Priyansh Jain
…On Mon, 5 Oct 2020, 02:11 Thiago Lins, ***@***.***> wrote:
Hm, great. Maybe I could migrate this to TypeScript :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#6 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AGJHFR6LEDWUAYYED76J3YLSJDMXZANCNFSM4SDGX2TQ>
.
|
All the backend code is in a single file! This stinks. Here's your chance to do some cool refactoring and celebrate Hacktoberfest! The code is fully functional, all you have to do is run a linting program and move code around. If you have tons of time, you can migrate it to TypeScript as well!
The text was updated successfully, but these errors were encountered: