-
Notifications
You must be signed in to change notification settings - Fork 18
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
[Labour#1]: Port wimp to use hercules #17
Comments
I've already opened an issue metakgp/wimp#39 4 days ago and was waiting for someone to take it. :D I can't think of myself doing that. It'll delete most of my code :( |
Yes, I agree with the fact that most of the scrapping logic in |
Will definitely work on this sometime, then @kshitij10496 :) |
@themousepotato Getting to delete old code and write new, better code is always a privilege! In these projects, we have the time and resources to re-write stuff well, that's a great opportunity. I often learn a lot about the mistakes I made the first time around whenever I do a re-write of something I made (eg: bar-horizontal and cli-cube-timer, I wrote both before I knew what Node was really for, and after doing Node for a couple years, I re-wrote them! Couldn't believe the kind of Node I used to write!) If you get the chance to, you should definitely be the one to re-write wimp! |
Sure @icyflame :D |
@themousepotato Can you experiment with using
hercules
API to powerwimp
?Given that the endpoints currently have bare minimum documentation, I would be happy to assist you. 😄
cc @icyflame @amrav
The text was updated successfully, but these errors were encountered: