Skip to content

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

Merged
kennethreitz merged 2 commits into
realpython:masterfrom
six8:todo
Dec 30, 2011
Merged

Mostly added todos, AUTHORS.rst, fixed some links, added some text editors#46
kennethreitz merged 2 commits into
realpython:masterfrom
six8:todo

Conversation

@six8
Copy link
Copy Markdown
Contributor

@six8 six8 commented Dec 30, 2011

No description provided.

@six8
Copy link
Copy Markdown
Contributor Author

six8 commented Dec 30, 2011

I accidentally the whole s.

Comment thread docs/starting/dev-env.rst
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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