Skip to content
This repository was archived by the owner on May 14, 2024. It is now read-only.

Veli Ataseven JavaScript Homework Week 1 - #109

Closed
veliataseven wants to merge 4 commits into
HackYourFuture:masterfrom
veliataseven:master
Closed

Veli Ataseven JavaScript Homework Week 1#109
veliataseven wants to merge 4 commits into
HackYourFuture:masterfrom
veliataseven:master

Conversation

@veliataseven

Copy link
Copy Markdown

Veli Ataseven JavaScript Homework Week 1

@marcianoviereck marcianoviereck left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Really nice solutions! Cool functions,
approved!


let highest;

if(a > z)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nice solution! You can also use math.max. But this is a fine solution!

'num2'
];

for(let i = 0; i < (variables.length - 1); i++)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

wow, advanced solution! Really nice!

Comment thread Week2/week1/9_modulo.js
@@ -0,0 +1,12 @@
for(let x = 1; x < 8; x++)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

cool that you know how to use loops already! Will help a lot in future courses

@wouterkleijn

Copy link
Copy Markdown
Contributor

Thanks for submitting the homework and trying to do some cool stuff! @veliataseven

@veliataseven

Copy link
Copy Markdown
Author

Thanks for submitting the homework and trying to do some cool stuff! @veliataseven

You are welcome.. I am trying to do somethings that as much as possible I can

@NoerGitKat NoerGitKat closed this Jun 2, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants