Thanks to visit codestin.com
Credit goes to github.com

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

test pr homework from unmesh #357

Closed
wants to merge 1 commit into from

Conversation

unmeshvrije
Copy link

Review

Copy link

@hasan-sh hasan-sh left a comment

Choose a reason for hiding this comment

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

Tests pass

// Replace this comment and the next line with your code
console.log(tasks, hourlyRate);
return tasks
.map(task => task.duration / 60)

Choose a reason for hiding this comment

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

Very nice chain :)

@@ -1,8 +1,7 @@
'use strict';

function doubleOddNumbers(numbers) {
// Replace this comment and the next line with your code
console.log(numbers);
return numbers.filter(number => number % 2 !== 0).map(number => number * 2);
Copy link

@hasan-sh hasan-sh May 31, 2019

Choose a reason for hiding this comment

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

Is the name number in the map method the best name you could give??

Think about what numbers you're filtering out!!

@remarcmij remarcmij closed this Jul 9, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants