algorithm: reverse#1197
Conversation
appgurueu
left a comment
There was a problem hiding this comment.
Please...
- Name this
reverse - Use a proper JSDoc comment
- Why is your comment screaming at me?
- Don't use a
const Arrayin your tests (shadows the built-inArray) - just inline the arrays - Consider using
eachhere - Declare
tempasconstinside the loop
Just to be sure:
Sorry for these mistakes, I'm a beginner student that heard and is learning about the open-source software |
Not only the file names, also the function name.
Sorry, I should've made myself more clear. I did not mean "use for-each for reversing". I meant use Jest's |
document change: import from ../Reverse.js
Everything is done, I guess. |
appgurueu
left a comment
There was a problem hiding this comment.
You forgot to rename the files...



Describe your change:
Algorithm to invert elements in an array:
[a,b,c,d] //output [d,c,b,a]
Checklist:
Example:
UserProfile.jsis allowed butuserprofile.js,Userprofile.js,user-Profile.js,userProfile.jsare notFixes: #{$ISSUE_NO}.