[beta] Clarify title for "Basic Data Structures: Copy an Array with slice()"
See original GitHub issueWelcome first-time open source contributors! 🏆
Challenge copy-an-array-with-slice has an issue.
The title says “Copy an Array with slice()”, but the challenge could be better described as “Copy Array items using slice()” as Array.slice()
not only can copy whole arrays, but can copy individual items as well.
If you want to help fixing this, the contribution guidelines can help you get started.
This is the line where you can make this change.
Feel free to come chat with us in the Contributors chat room if you have any questions - or just want to say hi!
Good luck & Happy coding! 😄 👍
Already resolved:
✅ <strike>1. The test for this challenge does not run properly. The console gives me plenty of “Minified React Error #32”, Similar to the message reported in #12656.</strike>
✅ <strike>2. I originally thought the second test case just had a problem with formatting, but since there’s nothing wrong with it, I suspect it has to do with the React error. However, I noticed that the second test could be improved with a cleaner assertion.
I suggest this line gets replaced with
"assert(/\\.slice\\(/.test(code), 'message: The <code>forecast</code> function should utilize the <code>slice()</code> method');"
</strike>
Issue Analytics
- State:
- Created 7 years ago
- Comments:10 (9 by maintainers)
Top GitHub Comments
@systimotic Haha looks like you’re faster than me! 😉
@systimotic This would be a good case for first-timers-only 😊
I don’t think we need a new issue - I’ll just edit the original post and clarify what is left to be done.