-
Notifications
You must be signed in to change notification settings - Fork 13
-
Notifications
You must be signed in to change notification settings - Fork 13
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
Set up a virtual environment #4
Comments
I'll start with this, this seems easy. :| |
Great! Mention me if you run into any issues. |
@abinashmeher999 I have made the environment. Just to be sure that I did that correctly, can you tell me how does one commit a virtual environment? Also, what next? |
This is the fun part. You can't commit all of the virtual environment. So it would be great if you could write a script(bash script would do) that creates the virtual environment and then prints the instructions to change into it. Also you will need to freeze dependent packages into requirements.txt using |
Yes I've already created the requirements.txt file. Actually, that was the only new addition to the project after creating virtual environment as all others are in .gitignore. |
No not a makefile. A simple |
@ayuhsya I see that you have committed the changes on the master branch itself. It is a bad practice to do so. Say you made all your changes to the master branch, you send the PR and let's assume didn't get merged because of some issues. Now you want to work on top of the latest master. Where do you pull that branch to? I hope you understand. |
@abinashmeher999 can I setup virtual environment in python 2.7 I am very much familiar with this. |
@harshcrop We will be using Python 3 in the future because since we are dealing with wikipedia here and there will lots of unicode characters. And handling unicode is lot less painful in Python 3. Besides making a virtualenv for Python 3 is not very different from how you would do that for Python 2.7. @ayuhsya Has already completed doing it. It would be pointless to the same thing again. Pull @ayuhsya's commits from his master into a new branch and see if you can complete the PR. |
@abinashmeher999 okay make new branch again can check his repo |
Sorry didn't get you. |
@abinashmeher999 I am asking that what I have to do now if this task is completed so, can I set up virtual environment for me in my repo and push into my branch and start working on new issue. That what I am asking you?? |
Yeah you can go ahead. Try #3 |
okay doing it |
@abinashmeher999 Yes I know that we don't work on the master branch. But this task wasn't that major so I didn't bother creating a new branch. 😐 |
@ayuhsya You will realise that even a small task doesn't justify working on the master once you are on your second PR(not first) to this repo. Because the work needed then just to clean up is more than just creating a new branch. It's a small change, so no worries 😄. Better clean it up before it becomes a headache. Maybe you can move those commits to a new branch. You can google it on how to do that. It's not uncommon to commit on your master by mistake. |
Okay, I'll do things on new branch now. But what do I do to send pull request/what else is left to do in this issue? |
Addressed in #11. |
An isolated environment to which developers can switch and then start contributing. I would prefer Python 3 environment. This might also require minor refactoring of the code too.
The text was updated successfully, but these errors were encountered: