Skip to content
This repository was archived by the owner on Sep 8, 2024. It is now read-only.

Simplify return; since it is already implied#15

Open
Elaniobro wants to merge 1 commit intoIamManchanda:masterfrom
Elaniobro:reverseInt-refactor
Open

Simplify return; since it is already implied#15
Elaniobro wants to merge 1 commit intoIamManchanda:masterfrom
Elaniobro:reverseInt-refactor

Conversation

@Elaniobro
Copy link
Copy Markdown

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!!

@IamManchanda
Copy link
Copy Markdown
Owner

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

@Elaniobro
Copy link
Copy Markdown
Author

@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.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants