Skip to content

Mostly added todos, AUTHORS.rst, fixed some links, added some text editors #46

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

Merged
merged 2 commits into from
Dec 30, 2011

Conversation

six8
Copy link
Contributor

@six8 six8 commented Dec 30, 2011

No description provided.

@six8
Copy link
Contributor Author

six8 commented Dec 30, 2011

I accidentally the whole s.

::

$ pip install ipython
pip install ipython
Copy link
Contributor

Choose a reason for hiding this comment

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

$ should be there.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

OK, I wasn't sure. Some examples had them, some didn't. I prefer not to have them because it makes it easier to copy and paste commands without it, however taking the stance of using $ for prompt and >>> for python shell will make it more clear what the code is for. I'll start a styleguide for contributing to the docs so there is more consistency from the contributors.

Copy link
Contributor

Choose a reason for hiding this comment

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

That would be great!

  • $ for prompt.
  • >>> for code that is being actively interpreted (in a repl)
  • just code for python code examples

kennethreitz pushed a commit that referenced this pull request Dec 30, 2011
Mostly added todos, AUTHORS.rst, fixed some links, added some text editors
@kennethreitz kennethreitz merged commit 6a4ed7a into realpython:master Dec 30, 2011
kennethreitz pushed a commit that referenced this pull request Dec 30, 2011
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.

2 participants