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

Add a development environment setup script #74

Closed
wants to merge 2 commits into from

Conversation

ChakshuGupta
Copy link
Collaborator

Installs mongodb
Installs python requirements

Installs mongodb
Installs python requirements
@dhirensr
Copy link
Collaborator

@ChakshuGupta : Thank you for your interest and PR.
could you please explain how would the script be useful for the repository?

@ChakshuGupta
Copy link
Collaborator Author

Hi,

It would be useful to have the installation of all the required packages in one script. It helps with anyone wanting to contribute as it eases up the process of setting up the required development environment.

@dhirensr
Copy link
Collaborator

dhirensr commented Mar 15, 2020

@ChakshuGupta : I agree that this would be useful script to install all the dependencies. However it's OS specific and wouldn't work on all Operation Systems.
Hence I think we would not integrate this for now.
you could work on other tasks like #67 , #69
@Ali-Razmjoo : what do you think?

@ChakshuGupta
Copy link
Collaborator Author

Okay! Thank you! I'll start working on other tasks!

@Ali-Razmjoo
Copy link
Collaborator

Hi, thanks for the PR,

actually if you can convert this script to a dockerfile, it will solve this card ... https://github.com/zdresearch/OWASP-Honeypot/projects/1#card-22046785

Bests.

@ChakshuGupta
Copy link
Collaborator Author

Hi,

Great! I'll get on it.

@dhirensr
Copy link
Collaborator

@ChakshuGupta : should you need any help just send a PR and I can help. 😄

@ChakshuGupta
Copy link
Collaborator Author

Thanks. I have sent it in a PR.

@Ali-Razmjoo
Copy link
Collaborator

is this PR migrated to #75?

@dhirensr
Copy link
Collaborator

dhirensr commented Apr 8, 2020

@Ali-Razmjoo : yes. I guess we can close this PR.

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.

None yet

3 participants