second pass edits

Author: grobergm

config.yml

@@ -72,12 +72,18 @@ steps:
       - type: gate
         left: '%payload.comment.body%'
         operator: <
-        right: 6
+        right: 3
         else:
         - type: respond
           with: 03-unlucky.md
           data: 
             rollNum: '%payload.comment.body%'
+        - type: respond
+          with: 03-input.md
+      - type: respond
+        with: 03-lucky.md
+        data: 
+          rollNum: '%payload.comment.body%'
       - type: respond
         with: 03-input.md
         issue: 3
@@ -93,5 +99,4 @@ steps:
         with: 03-complete.md
         issue: 3
       - type: closeIssue
-        issue: 3
-
+        issue: 3

responses/02-multiple.md

@@ -34,7 +34,7 @@ import random
 dice_rolls = 2
 for i in range(0,dice_rolls):
   roll = random.randint(1,6)
-  print(f'You rolled a {roll}')`
+  print(f'You rolled a {roll}')
 ```
 
 In most game situations, rolling multiple dice is followed by adding their values together. Let's do that now. In Python, arithmetic can be performed on variables just by using the equivalent mathematical symbols. Want to add two variables together? It's as simple as: `variable_sum = variable_1 + variable_2`. Since our dice values are stored in a single updating variable instead of separate variables, however, we'll have to add the values upon themselves. First, make a variable called dice_sum above the `for` loop, and set it equal to 0:
@@ -59,6 +59,18 @@ It doesn't matter which version you prefer, just go with whichever way is easier
 ```python
 print('You have rolled a total of {dice_sum}')
 ```
-If you save and run the file, you'll now get the values of the two dice rolls and their combined sum. Neat!
+Make sure this statement is not indented (since that would include it in the for loop). Overall, your code should look like this:
 
-*[push your code]({{ repoUrl }}/issues/1) to GitHub to continue*
+```python
+import random
+dice_rolls = 2
+dice_sum = 0
+for i in range(0,dice_rolls):
+  roll = random.randint(1,6)
+  dice_sum += roll
+  print(f'You rolled a {roll}')
+print('You have rolled a total of {dice_sum}')
+```
+If you save and run the file, you'll now get the values of the two dice rolls and their combined sum. Now our code is developing some complexity!
+
+*[Push your code]({{ repoUrl }}/issues/1) to GitHub to continue*

responses/02-variables.md

@@ -14,9 +14,7 @@ We'll want to print that variable, too, so let's change the print function to re
 print(f'You rolled a {roll}')
 ```
 
-Run the code again, and it should say `You rolled a 5`. Nice! Now we're getting a number. Go ahead and `push` your code with the output as the comment within `commit` to move on. 
-
-A die that only rolls 5's isn't useful though, so let's add in the randomness that dice are used for.
+Run the code again, and it should say `You rolled a 5`. Nice! Now we're getting a number. A die that only rolls 5's isn't useful though, so let's add in the randomness that dice are used for.
 
 
 ## Randomizing Variables

responses/03-conditionals.md

@@ -1,20 +1,6 @@
-## Adding Conditionals
-Now our code is developing some complexity! By adding our variable manipulation into our `for` loop, the code now looks like this:
-
-```python
-import random
-dice_rolls = 2
-dice_sum = 0
-for i in range(0,dice_rolls):
-  roll = random.randint(1,6)
-  dice_sum += roll
-  print(f'You rolled a {roll}')
-print('You have rolled a total of {dice_sum}')`
-```
-
 Let's add some flavor to our responses, dependent on how we roll. We can do this by using conditionals. We'll put a command that only triggers if a certain condition is met. For example, let's add "Critical Fail" in the printed statement if the die roll is a one.
 
-In Python, we can set this as the criteria by replacing our print statement with:
+In Python, we can set this as the criteria by replacing our print statement in the loop with:
 
 ```python
 if roll == 1:
@@ -37,4 +23,22 @@ elif roll == 6:
   print(f'You rolled a {roll}! Critical Success!')
 ```
 
+With the conditionals added, your code should look something like this:
+
+```python
+import random
+dice_rolls = 2
+dice_sum = 0
+for i in range(0,dice_rolls):
+  roll = random.randint(1,6)
+  dice_sum += roll
+  if roll == 1:
+    print(f'You rolled a {roll}! Critical Fail')
+  elif roll == 6:
+    print(f'You rolled a {roll}! Critical Success!')
+  else:
+    print(f'You rolled a {roll}')
+print(f'You have rolled a total of {dice_sum}')
+```
+
 Leave a comment with the *number of rolls* it took to get a Critical Success. 

responses/03-input.md

@@ -1,23 +1,3 @@
-Wow, it only took {{ rollNum }} rolls? 🤩 You must be lucky! 🍀
-
-With the conditionals added, your code should look something like this:
-
-```python
-import random
-dice_rolls = 2
-dice_sum = 0
-for i in range(0,dice_rolls):
-    roll = random.randint(1,6)
-    dice_sum += roll
-    if roll == 1:
-        print(f'You rolled a {roll}! Critical Fail')
-    elif roll == 6:
-        print(f'You rolled a {roll}! Critical Success!')
-    else:
-        print(f'You rolled a {roll}')
-print(f'You have rolled a total of {dice_sum}')
-```
-
 We have a working dice roller right now! It's useful for a very specific situation: rolling two six-sided dice. What if we want to change the number of dice we roll? We can manually edit the code each time, but there's a better way to do this: adding a user `input`.
 
 Instead of having `dice_rolls` set to a specific value, let's set it equal to:
@@ -33,12 +13,35 @@ Tabletop games use dice with all number of sides so it'll be useful to add a use
 dice_size = int(input('How many sides are the dice?'))
 ```
 
-When changing a value we had assumed to be a specific number previously, we need to make sure the rest of our code still makes sense. All dice start with one, so our `if` statement regarding one still makes sense. Our `elif` statement regarding six doesn't though, as it is no longer guaranteed to be the highest number. Let's make it so our "Critical Success" line is printed to whatever the highest number of our desired dice is. The highest number a dice can roll is the same as the number of sides, so we just to replace our `elif` statement with:
+Change the max value of our roll like this
+```python
+roll = random.randint(1,dice_size)
+```
+
+When changing a value we had assumed to be a specific number previously, we need to make sure the rest of our code still makes sense.  All dice start with one, so our `if` statement regarding one still makes sense. Our `elif` statement regarding six doesn't though, as it is no longer guaranteed to be the highest number. Let's make it so our "Critical Success" line is printed to whatever the highest number of our desired dice is. The highest number a dice can roll is the same as the number of sides, so we just to replace our `elif` statement with:
 
 ```python
 elif roll == dice_size:
 ```
 
 Try it out! Now you can manually input the number of dice and number of sides on the dice knowing that the `Critical Success!` line will realign accordingly. 
 
-*[push your code]({{ repoUrl }}/issues/1) to GitHub to finish the course!*
+Your final code should look like this:
+
+```python
+import random
+dice_rolls = int(input('How many dice would you like to roll?'))
+dice_size = int(input('How many sides are the dice?'))
+dice_sum = 0
+for i in range(0,dice_rolls):
+  roll = random.randint(1,dice_size)
+  dice_sum += roll
+  if roll == 1:
+    print(f'You rolled a {roll}! Critical Fail')
+  elif roll == dice_size:
+    print(f'You rolled a {roll}! Critical Success!')
+  else:
+    print(f'You rolled a {roll}')
+print('You have rolled a total of {dice_sum}')
+```
+*[Push your code]({{ repoUrl }}/issues/1) to GitHub to finish the course!*

responses/03-lucky.md

@@ -0,0 +1 @@
+Wow, it only took {{ rollNum }} rolls? 🤩 You must be lucky! 🍀

responses/03-unlucky.md

@@ -1,44 +1 @@
-Wow, it took {{ rollNum }} rolls? 😮 That is {{ rollNum/6 }} longer than it should have taken! Oh well, you don't need good luck if you have crazy python skills! 😎 🐍
-
-With the conditionals added, your code should look something like this:
-
-```python
-import random
-dice_rolls = 2
-dice_sum = 0
-for i in range(0,dice_rolls):
-    roll = random.randint(1,6)
-    dice_sum += roll
-    if roll == 1:
-        print(f'You rolled a {roll}! Critical Fail')
-    elif roll == 6:
-        print(f'You rolled a {roll}! Critical Success!')
-    else:
-        print(f'You rolled a {roll}')
-print(f'You have rolled a total of {dice_sum}')
-```
-
-We have a working dice roller right now! It's useful for a very specific situation: rolling two six-sided dice. What if we want to change the number of dice we roll? We can manually edit the code each time, but there's a better way to do this: adding a user `input`.
-
-Instead of having `dice_rolls` set to a specific value, let's set it equal to:
-```python
-int(input('How many dice would you like to roll?'))
-```
-
-Running the code will give a prompt with that question and we can input our desired number of dice. One thing to note: the value a user inputs defaults to a `str` type and we need it to be a whole number, or an `int` in Python; that's why we encompass the `input` command in an `int()`. Try it out!
-
-Tabletop games use dice with all number of sides so it'll be useful to add a user input for that too. Below where you set `dice_rolls` add this line:
-
-```python
-dice_size = int(input('How many sides are the dice?'))
-```
-
-When changing a value we had assumed to be a specific number previously, we need to make sure the rest of our code still makes sense. All dice start with one, so our `if` statement regarding one still makes sense. Our `elif` statement regarding six doesn't though, as it is no longer guaranteed to be the highest number. Let's make it so our "Critical Success" line is printed to whatever the highest number of our desired dice is. The highest number a dice can roll is the same as the number of sides, so we just to replace our `elif` statement with:
-
-```python
-elif roll == dice_size:
-```
-
-Try it out! Now you can manually input the number of dice and number of sides on the dice knowing that the `Critical Success!` line will realign accordingly. 
-
-*[push your code]({{ repoUrl }}/issues/1) to GitHub to finish the course!*
+Wow, it took {{ rollNum }} rolls? 😮 That is {{ rollNum/3 }} times longer than it should have taken! Oh well, you don't need good luck if you have crazy python skills! 😎 🐍