Skip to content

code review#26

Open
ersel wants to merge 2 commits intoCommandShiftHQ:mainfrom
ro8inro8in:master
Open

code review#26
ersel wants to merge 2 commits intoCommandShiftHQ:mainfrom
ro8inro8in:master

Conversation

@ersel
Copy link

@ersel ersel commented Nov 14, 2020

No description provided.


//return String(number).split("").reverse().join("");
let num = 12345,
arr = String(num).split("").reverse().map(Number);
Copy link
Author

Choose a reason for hiding this comment

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

you should have used the number argument here, rather than defining the number yourself


const humanCatDogYears = number => {

if (number === 1) {
Copy link
Author

Choose a reason for hiding this comment

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

you're missing the logic... what if I wanted to see the calculation when number is 4,5,6,7?


describe("reachDestination", () => {
test("returns string with estimated time of arrival", () => {});
xit ( "returns array of human years ",()=>{
Copy link
Author

Choose a reason for hiding this comment

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

not attempted?

Copy link
Author

@ersel ersel left a comment

Choose a reason for hiding this comment

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

similar problems to previous exercise

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