Add more documentation in ListViewDataSource

[umhan35]

ListViewDataSource's default data extractor can actually expect another data form:

{ sectionID_1: [ <rowData1>, <rowData2>, ... ], ... }

[sahrens]

@facebook-github-bot import

[facebook-github-bot]

Thanks for importing. If you are an FB employee go to https://our.intern.facebook.com/intern/opensource/github/pull_request/1593093057609165/int_phab to review.

[sahrens]

Thanks for the fix!

In the future, please make sure to clean up trailing spaces and any other lint.

[umhan35]

Sorry, I just added something in the comment. Why do we need to clean up trailing spaces and any other lint in this context?

[sahrens]

code is code and lint is lint, whether it's a comment or not.

[umhan35]

OK. I just noticed that there are trailing spaces. I didn't mean it and it's hard to find. Sorry.

[sahrens]

Some editors can be configured to automatically remove trailing whitespace which is nice. Does running lint pick up these issues for you?

On May 15, 2015, at 5:11 PM, Zhao Han notifications@github.com wrote:

OK. I just noticed that there are trailing spaces. I didn't mean it and it's hard to find. Sorry.

—
Reply to this email directly or view it on GitHub.

[umhan35]

Oh, I edited the file using GitHub's edit button because I think the change is minor... (My IDE, WebStorm, for my react native project has the automatic trailing space removal feature.)

I'm new to node.js and lint is also new to me. I just figured out myself: the ESLint project (http://eslint.org/). But how do you run it? I tried several commands and they don't work.

[umhan35]

Thank you @sahrens! Just saw the update of CONTRIBUTING.md