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

Sully Birashk #164

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

Sully Birashk #164

wants to merge 8 commits into from

Conversation

SullyBirashk
Copy link

No description provided.

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 Sully, I just had a chance to go over 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.

@@ -4,40 +4,75 @@

# Build a Bear

# Creating a function called build_a_bear with 5 Arguments called name, age, fur,
# clothes, and special_power.

Choose a reason for hiding this comment

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

These are called parameters. Arguments are the values that you actually pass into the method (i.e. on lines 38 and 40). Parameters and arguments are 2 sides of the same coin, so to speak.


# Your function should include an if/else statement that meets the following criteria
if danger_level < 50

Choose a reason for hiding this comment

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

What if danger_level == 50 ?

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