algorithms-javascript icon indicating copy to clipboard operation
algorithms-javascript copied to clipboard

Simplify return; since it is already implied

Open Elaniobro opened this issue 6 years ago • 2 comments

Documentation: ✍️

Keeping in line with the rest of your coding style, this solution could be refactored to use the implied return statement.

Documentation: 📄

Reference: 🔍

p.s. Awesome work, thank you for sharing!!

Elaniobro avatar Aug 18 '18 03:08 Elaniobro

Looks good to me ... I am outside my workflow right now... but when I get chance will test (as a process) and merge it :)

IamManchanda avatar Aug 20 '18 08:08 IamManchanda

@IamManchanda sounds good, I also ran the test :)

   reverseInt-refactor  jest 003-reverseint/test.js                                                                ✔  2265  11:05:37 
 PASS  003-reverseint/test.js
  ✓ ReverseInt function exists (6ms)
  ✓ ReverseInt handles 0 as an input (3ms)
  ✓ ReverseInt flips a positive number (2ms)
  ✓ ReverseInt flips a negative number (2ms)

Test Suites: 1 passed, 1 total
Tests:       4 passed, 4 total
Snapshots:   0 total
Time:        5.484s
Ran all test suites matching /003-reverseint\/test.js/i.

Elaniobro avatar Aug 20 '18 15:08 Elaniobro