Skip to content
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

Remove extra comma at the end of stringifySafe call #1309

Closed
wants to merge 1 commit into from

Conversation

sdd330
Copy link

@sdd330 sdd330 commented May 16, 2015

No description provided.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label May 16, 2015
@ide
Copy link
Contributor

ide commented May 16, 2015

I think these are intentional. d9b7e63

@sdd330
Copy link
Author

sdd330 commented May 16, 2015

Ok, got it

@sdd330 sdd330 closed this May 16, 2015
@victorteokw
Copy link

What does this for? Why adding trailing comma?

@sdd330
Copy link
Author

sdd330 commented May 16, 2015

For testing purpose maybe

@ide
Copy link
Contributor

ide commented May 16, 2015

It's so that multiline function calls affect just one line of code if you add or remove an argument. Same thing as array and object literals.

ayushjainrksh pushed a commit to MLH-Fellowship/react-native that referenced this pull request Jul 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants