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

Restructure the API code and make improvements. #411

Open
agentmilindu opened this Issue Jan 14, 2018 · 4 comments

Comments

Projects
None yet
5 participants
@agentmilindu
Member

agentmilindu commented Jan 14, 2018

Study the code and identify the issues, bad coding decisions, security vulnerabilities, anti-patterns, etc.
Then identify the best way to restructure the code and fix the issues above while optimizing the code and increasing readability and maintainability.

Propose us all the issues you see and the changes you plan to do.

@raju249

This comment has been minimized.

raju249 commented Feb 13, 2018

I can take up this issue, will come up with a plan after throughly understanding what Bassa does.
Would like to make this project modularised. With each module having files only which are related to that particular Module.

Also we can add a documentation wiki showing what each module does.

@raju249

This comment has been minimized.

raju249 commented Feb 13, 2018

Can I work on this directly or I will first have to pass the proposal and start working when GSoC starts ?
@agentmilindu @charithccmc

@saisumit

This comment has been minimized.

saisumit commented Feb 22, 2018

Can you point to the part of code, where you think restructuring can be done to give a head start.

@vishrutkohli

This comment has been minimized.

vishrutkohli commented Mar 8, 2018

Hi, i am really excited to work on this issue. To start contributing to this issue should I first make the proposal or just start with small enhancements to contribute . for example writing the python code in pep-8 style.

thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment