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.

Sign up for GitHub

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

Jump to bottom

Alphabetical sort produces random order #838

Closed
wopian opened this issue Jul 6, 2017 · 1 comment
Closed

Alphabetical sort produces random order #838

wopian opened this issue Jul 6, 2017 · 1 comment
Assignees
@anthony-redFox
Labels
bug

Comments

@wopian
Copy link

wopian commented Jul 6, 2017

Changed docs to use --sort-order alpha which produced an unexpected ordering with 4.0.0-rc.1

Input code with documentation build src/index.js -f md -o DOCS.md --sort-order alpha

Output ToC (default, source) 👌
Output ToC (--sort-order alpha) 🤔

Node: 7.10
OS: Win10
@anthony-redFox
Copy link
Member

@wopian Thanks, it is a bug. I will take a look.

@anthony-redFox anthony-redFox self-assigned this Aug 3, 2017
anthony-redFox added a commit to anthony-redFox/documentation that referenced this issue Aug 3, 2017
@anthony-redFox
Removed sort by memberof, when the memberOf is the same of items then…
f1042fc 
… sort function doesn't works correctly.

Added Unit Tests
Fixed documentationjs#838
anthony-redFox added a commit to anthony-redFox/documentation that referenced this issue Aug 7, 2017
@anthony-redFox
Removed sort by memberof, when the memberOf is the same of items then…
4edd3c7 
… sort function doesn't works correctly.

Added Unit Tests
Fixed documentationjs#838
@anthony-redFox anthony-redFox mentioned this issue Aug 7, 2017
Merged
@tmcw tmcw closed this as completed in #861 Aug 8, 2017
tmcw pushed a commit that referenced this issue Aug 8, 2017
@anthony-redFox @tmcw
fix: alphabetical sort and add flow notation to sort file (#861)
aa3496a 
* Removed sort by memberof, when the memberOf is the same of items then sort function doesn't works correctly.

Added Unit Tests
Fixed #838

* Flow.js : added flow notation for sort.js
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@anthony-redFox anthony-redFox

Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@wopian @anthony-redFox