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.

Sign up for GitHub

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

Jump to bottom

non graphical code. #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

non graphical code. #4

wants to merge 1 commit into from

Conversation

vartika-arora
Copy link

No description provided.

@hackrush01
Copy link
Contributor

Change the commit message to something more descriptive, for reference look here sympy PRs.
Add comments to your code.
Then PR

@hackrush01
Copy link
Contributor

Also your code is doing nothing when I just use python nf.py. Also give a more meaningful name to your file.

@vartika-arora
Copy link
Author

vartika-arora commented Feb 25, 2017 via email

@hackrush01
Copy link
Contributor

Check gitter

hackrush01
hackrush01 requested changes Feb 25, 2017
View reviewed changes
Copy link
Contributor

@hackrush01 hackrush01 left a comment

Choose a reason for hiding this comment

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

Merge the two commits to 1

@vartika-arora
The updated code and documentation
77db835 
new edited code for mastermind

Updated documentation of python-mastermind
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hackrush01 hackrush01 hackrush01 requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@vartika-arora @hackrush01