search

TheLethalCode / Artemis-arrow

14 stars 31 forks source link

Devel #24

Closed mridul2899 closed 5 years ago

mridul2899 commented 5 years ago

I have added the code written till now. It needs to be commented. It might also be needed to be formatted in a better manner. It would be nice if this could be merged with the devel branch so that others can also work upon it if they want to. Do suggest me the changes that you feel I must make before this code can get merged.

TheLethalCode commented 5 years ago

Ok, looks good. But since this mainly command line oriented, it would be best if you partition the program into functions because it will be easier to integrate. And do not keep author name as mandatory. Ho and btw, always try using if main == 'name' . https://stackoverflow.com/questions/419163/what-does-if-name-main-do

mridul2899 commented 5 years ago

Made the suggested changes. Ready to be merged.

TheLethalCode commented 5 years ago

LGTM :+1: The basic functionality of this has been taken care of. We will come back here, when we want to add some more features to it.