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

Lourdes Benites #201

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

Lourdes Benites #201

wants to merge 6 commits into from

Conversation

lourdesbnts
Copy link

No description provided.

Copy link

@mschae16 mschae16 left a comment

Choose a reason for hiding this comment

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

@lourdesbnts - Hi Lourdes, I just had a chance to go over your pre-work and you are officially Turing ready! I added some questions/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.


### Lesson Prep
You've learned a lot about how to take strong notes during Mod 0. Show us your skills while you learn how to pre-teach content for your first lesson in Mod 1!
- [ ] Complete the [Pre Teaching Practice exercise](https://gist.github.com/ericweissman/0036e8fe272c02bd6d4bb14f42fd2f79) gist
- [ ] Add a link to your gist here:
- [ ] Add a link to your gist here:
Copy link

Choose a reason for hiding this comment

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

Your pre-teaching practice gist appears to be missing.

}

changeProtein(newProtein) {
Copy link

Choose a reason for hiding this comment

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

Nice work here 💯

console.log('buzz');
//this will print of all previous conditions are false
Copy link

Choose a reason for hiding this comment

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

Thorough annotations here, nice job.

@@ -17,26 +20,35 @@ function buildABear(name, age, fur, clothes, specialPower) {

return builtBear
}

//arguments
buildABear('Fluffy', 4, 'brown', ['pants', 'jorts', 'tanktop'], 'give you nightmares');
Copy link

Choose a reason for hiding this comment

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

I would encourage you to be a little more specific here - lines 24 and 25 are invoking (or calling/executing) the buildABear function and passing in several arguments, for example.

// Print the last archEnemy to the console

console.log(archEnemies[2]);
Copy link

Choose a reason for hiding this comment

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

How would you print the last archEnemy to the console if you didn't know the length of the array?

// - badExcuse should be a string a hero would say if they are too afraid of the dangerLevel
var saveTheDay = "All done here, for now!";
var badExcuse = "I don't have enough strength for that";
Copy link

Choose a reason for hiding this comment

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

You don't need to declare saveTheDay and badExcuse as variables - they're just parameters in the function - essentially placeholders for whatever arguments get passed in when the function is called or invoked.

maximizeEnergy() {
this.energyLevel = 1000;
}
gainPower(number) {
Copy link

Choose a reason for hiding this comment

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

Nice!

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