-
-
Notifications
You must be signed in to change notification settings - Fork 5.7k
Created FlattenedArray.js #781
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
Flatten Array using recursive method
This pull request introduces 1 alert when merging a7477cc into 5f601fa - view on LGTM.com new alerts:
|
This pull request introduces 1 alert when merging 228a5d3 into 5f601fa - view on LGTM.com new alerts:
|
return res; | ||
} | ||
|
||
module.exports = flattened; |
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.
module.exports = flattened; | |
export { flattened } |
Use ESM
This pull request introduces 2 alerts when merging 4ba6a1b into 8b27316 - view on LGTM.com new alerts:
|
Recursive/FlattenedArray.js
Outdated
|
||
// JEST TEST | ||
|
||
const flattened = require('./FlattenedArray'); |
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.
You have to put this in a new file. Also, use ESM.
Flatten Array using recursive method
Welcome to JavaScript community
Describe your change:
Checklist:
Example:
UserProfile.js
is allowed butuserprofile.js
,Userprofile.js
,user-Profile.js
,userProfile.js
are notFixes: #{$ISSUE_NO}
.