Final clean up of grammar and instructions on this your turn for recent edits.

Author: mikeckennedy

your-turn/04-refactoring/readme.md

@@ -6,19 +6,19 @@
 
 We are going to use a project located in the same folder as this file called `smelly_podcast` (as in code smells). This is code we used to download some things from Talk Python To Me.
 
-It has been contorted to have many problems or [code smells](https://en.wikipedia.org/wiki/Code_smell). We'll use refactoring to freshen this up.
+It has been _contorted_ to have many problems or [code smells](https://en.wikipedia.org/wiki/Code_smell). We'll use refactoring to freshen this up.
 
-**Note**: Before you first run this app, you'll need to make sure you have `requests` installed via pip.
+**Note**: Before you first run this app, you'll need to make sure you have a virtual environment active and `requests` installed via pip.
 
 ## Large method *smell*
 
-Look at `program.py` and notice this somewhat complex app is actually just one big method, yuck. We'll use several refactoring to fix this.
+Look at `program.py` and notice this somewhat complex app is actually just one big method, yuck. We'll use several refactorings to fix this.
 
 First, notice the section with the comment **"SHOW THE HEADER"**.
 
 Take that small piece of code and make it its own method. Highlight the lines, choose **Refactor > Refactor this**.
 
-Choose method and name it `show_header()`. Notice it was created below and called from your current location.
+Choose method and name it `show_header`. Notice it was created below and called from your current location.
 
 After each refactoring, run the program to make sure it still works!
 
@@ -45,7 +45,7 @@ Accomplish this by refactoring the `url` variable and choosing *inline variable*
 -----------------------
 
 
-Next up, we have **"GET SHOW ID RANGE"**. We want a method that will return both the `latest_show_id` as well as the `oldest_show_id`. This is interesting because you'll see that PyCharm knows about tuple packing/unpacking on method calls. Highlight both:
+Next up, we have the **"GET SHOW ID RANGE"** section. We want a method that will return both the `latest_show_id` as well as the `oldest_show_id`. This is interesting because you'll see that PyCharm knows about tuple packing/unpacking on method calls. Highlight both:
 
 ```python
 latest_show_id = max(episode_data.keys())
@@ -80,7 +80,7 @@ Let's move the data container and type first. Highlight these two lines and "mov
 	Episode = namedtuple('Episode', 'title link pubdate show_id')
 	episode_data = {}  
 
-**Note**: If you have **not** opened just the `smelly_podcast` folder as the project root, you will need to market the folder `smelly_podcast` as a **sources root** for that to work. It should be blue like this:
+**Note**: If you have **not** opened just the `smelly_podcast` folder as the project root, you will need to market the folder `smelly_podcast` as a **sources root** for that to work. It should be **blue** like this:
 
 ![](./sources-root.png)
 
@@ -125,7 +125,7 @@ def main():
 
 ## Variable absence *smell*
 
-Variable absence is something I just made up. There doesn't seem to be an agreed upon smell for it, but I'm sure you've seen it. Here's the deal. What does this mean:
+**Variable absence** is a smell I created. There doesn't seem to be an agreed upon smell for it, but I'm sure you've seen it. Here's the deal. What does this mean:
 
 `result = 7 * max(inputs) * (match / count)`
 
@@ -134,14 +134,14 @@ Let's try this one instead.
 ```python
 largest_cell = max(inputs)
 ratio = (match / count)
-DICE_MULTIPLIER= 7
+DICE_MULTIPLIER = 7
 
 result = DICE_MULTIPLIER * largest_cell * ratio
 ```
 
-Well, you'll still have to use your imagination. But it seems to do with a board game, rolling dice, etc. Why is one clear and the other isn't? The expression is computing many values inline. Once named, they are much clearer what they are and how they are combined.
+Well, you'll still have to use your imagination. But it seems to do with a board game, rolling dice, etc. Why is one clear and the other isn't? The expression is computing many values inline all at the same time. Once named, the pieces are much clearer.
 
-Open `service.py`. Notice this line `dom.findall('channel/item')` appears multiple times. It represents the `items` from the RSS feed. 
+Open `service.py`. Notice this line `dom.findall('channel/item')` appears multiple times. It represents the `items` elements from the RSS feed (part of the RSS 2.0 spec). 
 
 Make a variable for it by highlighting either single expression, choose refactor > variable. Name it something like `items`. Choose 2 locations!