Wrong example in exercise

Programming JavaScript
1

In track JavaScript exercise Train Driver is:
getListOfWagons(1, 7, 12, 3, 14, 8, 5);
// => [1, 7, 12, 3, 14, 8, 3]
instead of:
getListOfWagons(1, 7, 12, 3, 14, 8, 5);
// => [1, 7, 12, 3, 14, 8, 5]

1 Like
2

Exercise link: Train Driver in JavaScript on Exercism

3

While we’re at it, I’m not certain but I think the @params on the first function is incorrect. @param {number[]} ids suggests the function takes a list, but the idea is to take a bunch of numbers?

1 Like
4

And another typo here.

-[..] an array there the first element
+[..] an array where the first element
1 Like
5

It used to not be possible to do this, but apparently now:

// @param {...number} ids

we’ll take a PR for all three changes!

6

I dont know about this, I know only about the issue in Instructions.

7

SleeplessByte says the track will take a PR :slight_smile: Do you want to PR all the changes? Just the one you pointed out? None?

8

No, sorry, I am busy. :slightly_frowning_face:

9

No worries. I’m happy to make the PR.

See Train Driver: fix typos and minor corrections by IsaacG · Pull Request #2685 · exercism/javascript · GitHub

1 Like
10

This has been merged so the issues should all be resolved. Thanks for the report!

1 Like