-
-
Notifications
You must be signed in to change notification settings - Fork 636
Added filles for the new rest and spread exericse #1989
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
Conversation
Dear meatball133Thank you for contributing to the JavaScript track on Exercism! 💙
Dear Reviewer/Maintainer
Automated comment created by PR Commenter 🤖. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for getting started with the content. I reviewed the instructions file and left some comments.
|
||
Your friend has been keeping track of how much each wagon weighs. Although they are not sure how many wagons and would like the data to be returned as an array. | ||
|
||
```exercism/note |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note blocks should not be used for the normal instructions of the exercise, only for special things that need to stand out (like the note at the top about using rest and spread).
Applies to all the tasks.
I feel like I have resolved the issues mentioned above. If not please tell me and I will give it another round |
Thank you for the work! We'll check it out. |
Could some one go through with me why errors are raised? The changes I made should not have caused this. |
Sorry I didn't see this. Seems you managed to solve this yourself! For future contributions, on this track repository you can post a comment with |
I see, no problem. But thanks for the tip. I made some commits to some other tracks, so I learnt there how the config file needs to be setup. |
I'll pick up the reviewing here :) |
So... I broke my back (spinal cord injury) because of stage IV cancer (including my spine), about 10 days before you pushed the last iteration, and forgot about it. I now have time to finish this up and merge it. @meatball133 the final changes look perfect. Only thing that needs to happen is a rebase on main, followed by a "sync" (so the |
Resolve #1500
This is the start for the exercise. There is still some stuff that I haven't started on like hints and design.
It is because I think they require that the rest is more or less finished.