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

doc: reorder fs #2432

Closed
wants to merge 1 commit into from
Closed

doc: reorder fs #2432

wants to merge 1 commit into from

Conversation

MattMS
Copy link

@MattMS MattMS commented Aug 19, 2015

Sort in alphabetical order but keep *sync with their related functions.
Classes follow functions.
Events follow methods in classes.

Sort in alphabetical order but keep *sync with their related functions.
Classes follow functions.
Events follow methods in classes.
@MattMS MattMS mentioned this pull request Aug 19, 2015
@brendanashworth brendanashworth added doc Issues and PRs related to the documentations. fs Issues and PRs related to the fs subsystem / file system. labels Aug 19, 2015
@Fishrock123
Copy link
Contributor

cc @nodejs/documentation

@MattMS
Copy link
Author

MattMS commented Sep 11, 2015

Should I close this?

@Trott
Copy link
Member

Trott commented Sep 11, 2015

I'm +1 on this in principle, although I'd rather have there be some general rule for the ordering of properties established by @nodejs/documentation and applied across all the docs. Absent that, though, something that has an ordering principle is better than something that is in arbitrary order.

@MattMS
Copy link
Author

MattMS commented Sep 11, 2015

For sure. I was thinking some headings may help and then alphabetical order within those.
Seems like this should be closed, with further discussion in #393.

@MattMS MattMS closed this Sep 11, 2015
@MattMS MattMS deleted the reorder-fs-doc branch September 11, 2015 00:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Issues and PRs related to the documentations. fs Issues and PRs related to the fs subsystem / file system.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants