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

Antonio Salmeron #167

Open
wants to merge 16 commits into
base: main
Choose a base branch
from
Open

Antonio Salmeron #167

wants to merge 16 commits into from

Conversation

amsalmeron
Copy link

Here is my finished M1 Prework!

Copy link

@corneliusellen corneliusellen left a comment

Choose a reason for hiding this comment

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

Hey Antonio, I did not see the mod 0 hero portion completed as part of the final_prep section. Can you please complete this ASAP and DM when completed?

@@ -4,10 +4,15 @@

# Build a Bear

# Create method (function) to build a bear with 5 arguments

Choose a reason for hiding this comment

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

These are called parameters, not arguments, when you define them in the method. Arguments and parameters are 2 sides of the same coin, so to speak.


# Your function should include an if/else statement that meets the following criteria
# - Danger levels that are above 50 are too scary for your hero. Any danger level that is above 50 should result in printing the bad_excuse to the terminal
# - Anything danger_level that is between 10 and 50 should result in printing the save_the_day string to the terminal
# - If the danger_level is below 10, it means it is not worth your time and should result in printing the string "Meh. Hard pass." to the terminal.

if danger_level > 50
puts bad_excuse
elsif danger_level > 10

Choose a reason for hiding this comment

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

What if danger_level == 10? "Meh. Hard pass" should only be printed if the danger_level is below 10.

Copy link

@corneliusellen corneliusellen left a comment

Choose a reason for hiding this comment

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

I just had a chance to go over the rest of your prework and you are officially turing ready! I added some comments to a few sections for areas to work on or adjustments to make before starting Mod 1. Keep up the hard work moving forward and let me know if you have any questions.

@amsalmeron
Copy link
Author

Sounds great. Thanks again, can't wait!

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