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

Give a try at #10 about Python 2 vs. 3. #25

Merged
merged 2 commits into from
May 18, 2015
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 5 additions & 2 deletions src/templates/index.html
Original file line number Diff line number Diff line change
Expand Up @@ -58,10 +58,13 @@ <h3 class="title">Hello World App</h3>
</div>
<div class="col-md-6">
<h3 class="title">Easy To Start</h3>
<p>Let's create a minimal web app in 5 minutes using Python 3.</p>
<p>Let's create a minimal web app in 5 minutes.</p>
<h4>1. Install Python</h4>
<p>Install the latest Python 3 from <a href="https://www.python.org/downloads/">Python.org</a>.</p>
<p>Install the latest Python from <a
Copy link
Member

Choose a reason for hiding this comment

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

Install the latest Python 2 or 3 from

href="https://www.python.org/downloads/">Python.org</a>.
Works fine under Python 2 or 3.</p>
Copy link
Member

Choose a reason for hiding this comment

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

Pyramid runs on both versions.

Copy link
Member Author

Choose a reason for hiding this comment

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

I was trying to be very, very terse on the wording in that section.

—Paul

On May 18, 2015, at 2:49 PM, Steve Piercy [email protected] wrote:

In src/templates/index.html #25 (comment):

       <h4>1. Install Python</h4>
  •      <p>Install the latest Python 3 from <a href="https://www.python.org/downloads/">Python.org</a>.</p>
    
  •      <p>Install the latest Python from <a
    
  •              href="https://www.python.org/downloads/">Python.org</a>.
    
  •      Works fine under Python 2 or 3.</p>
    
    Pyramid runs on both versions.


Reply to this email directly or view it on GitHub https://github.com/Pylons/tpc/pull/25/files#r30536364.

Copy link
Member

Choose a reason for hiding this comment

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

Maybe just remove it? The previous amended statement implies the same thing anyway.

Copy link
Member Author

Choose a reason for hiding this comment

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

The Python 3 part under "Install Pyramid" is due to the presence of pyvenv in the instructions.

We're over-talking this. Just merge whatever you'd like.

<h4>2. Install Pyramid</h4>
<p>For example, after installing Python 3:</p>
<pre class="nobs">
<code class="bash">$ pyvenv myproject
$ cd myproject
Expand Down