Skip to content
This repository has been archived by the owner on Sep 20, 2020. It is now read-only.

refractor using short circuit || operators #34

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

refractor using short circuit || operators #34

wants to merge 1 commit into from

Conversation

BrooksPatola
Copy link

@restyled-io
Copy link

restyled-io bot commented Sep 5, 2019

Hey there-

I'm a bot, here to let you know that some code in this PR might not
match the team's automated styling. I ran the team's auto-reformatting tools on
the files changed in this PR and found some differences. Those differences can
be seen in #35.

Please see that Pull Request's description for more details.

@restyled-io restyled-io bot mentioned this pull request Sep 5, 2019
Open
@BrooksPatola BrooksPatola mentioned this pull request Sep 5, 2019
Open
pranjaldatta
pranjaldatta reviewed Sep 5, 2019
View reviewed changes
Copy link
Collaborator

@pranjaldatta pranjaldatta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No significant improvements to the code.

@BrooksPatola
Copy link
Author

@pranjaldatta I thought the purpose of the issue was to refractor the code to take away un needed lines of code. That's what i did.

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

@pranjaldatta pranjaldatta pranjaldatta left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@BrooksPatola @pranjaldatta