-
Notifications
You must be signed in to change notification settings - Fork 120
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
Add MakeFile to ease development process #162
Comments
Hi @PradyumnaKrishna! Would you assign me this issue under GSSOC? |
Done @shreelakshmijoshi |
Hello, @PradyumnaKrishna I want work on this issue please assign me this issue under GSSOC...!! |
@dhruti29 , Please go ahead |
Hi @GouravSardana !
And dynamic-cli has all Python files. |
@PradyumnaKrishna is leading the effort from this side. Can you please explain what we need from our side ? |
oh okay, I just wanted to know what exact files should I include in the Make file to compile them? |
I agree with the point which you mentioned but let’s hear from @PradyumnaKrishna |
Makefile does lot more than just compiling, currently this repository has lot of commands that can be aggregated at one place. Let me give you an example all: setup lint test build
setup:
@echo SETUP
pip install -r requirements.txt
lint:
@echo LINT
# Linting it using linters like black, pylint etc
test:
@echo TEST
pytest
build:
@echo BUILD
...
publish:
@echo PUBLISH
...
... These all tasks and other similar tasks will get collected to one place. Currently we have to follow up the procedure to setup the development environment. After implementing Makefile, environment setup and package build will get done in one go, just have to type Note: #161 is work in progress which add poetry as dependency manager, the procedure to setup the development environment requires some changes after that. |
@PradyumnaKrishna, you're right. |
I would like to work on this issue, please assign this issue to me under GSSoC'23. |
🚀 Feature
Create a
MakeFile
to simplify project setup and ease the development process.It requires #161 to complete, so pickup that or wait for it to complete before start working on this.
Resources
Feel free to contact me for help.
The text was updated successfully, but these errors were encountered: