Skip to content
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

requirements.txt updated #155

Closed
wants to merge 2 commits into from
Closed

requirements.txt updated #155

wants to merge 2 commits into from

Conversation

nlok5923
Copy link
Member

@nlok5923 nlok5923 commented Sep 5, 2020

fixes: #142

@nlok5923 nlok5923 changed the title requirments updated requirements.txt updated Sep 5, 2020
@nlok5923
Copy link
Member Author

nlok5923 commented Sep 6, 2020

@iiitv/project-maintainers sir please review this pull request.

@mohitkyadav
Copy link
Member

@nlok5923 have you tested all the functionality?

@nlok5923
Copy link
Member Author

nlok5923 commented Sep 7, 2020

@mohitkyadav sir i hadn't tested all functionality i just use pip freeze command to get/update information about all the package installed in the project.

@aashutoshrathi
Copy link
Member

@nlok5923 The issue never said update dependencies and break the project. 🙂
Please make sure things work after you updated them.
PS: I think some of them need to be downgraded again to keep the project working.

cc: @mohitkyadav @iiitv/project-maintainers

@nlok5923
Copy link
Member Author

nlok5923 commented Sep 7, 2020

@nlok5923 The issue never said update dependencies and break the project. 🙂
Please make sure things work after you updated them.
PS: I think some of them need to be downgraded again to keep the project working.

cc: @mohitkyadav @iiitv/project-maintainers

@aashutoshrathi sir please can you help me how can i do this as i am new to django.

@mohitkyadav
Copy link
Member

@nlok5923 follow the instructions in the readme. Ask questions in slack if need help.

Copy link
Member

@theyashshahs theyashshahs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

are all these packages required for the project?

@aashutoshrathi
Copy link
Member

Doesn't look like that, one needs to check it.

@nlok5923 nlok5923 closed this Oct 15, 2020
@nlok5923 nlok5923 deleted the require branch October 15, 2020 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

update requirements.txt with latest versions of packages
4 participants