Sort functions by name in documentation #470
Conversation
hdwalters
force-pushed
the
sort-functions-by-name-in-docs
branch
from
c507a5b to
768abf6
Compare
|
I see also a little refactoring and probably this will require that #465 to be merged before this and updated or viceversa as they are both changing the |
|
Depends if someone approves the PR... |
There was a problem hiding this comment.
@hdwalters Tested it and it works. Found some small details that can help improve code readability!
hdwalters
force-pushed
the
sort-functions-by-name-in-docs
branch
from
460c612 to
831db27
Compare
hdwalters
force-pushed
the
sort-functions-by-name-in-docs
branch
from
831db27 to
d98f2af
Compare
|
So I tested with the script on amber-lang/amber-docs#39 Tests works but: Generates: @hdwalters I guess that doesn"t handle very good the output parameter |
Isn"t that incorrect @Mte90 ? shouldn"t that be |
|
No that line is extracted from an amber script, from the PR in amber-docs about stdlib |
|
@Mte90 could you then provide the entire script or at least some part of that script that shows the entire context? |
|
It was already in the comment amber-lang/amber-docs#39 |
Fixes #460. Sorts function declarations (we expect these to be the only top level elements in the abstract syntax tree) by name. Also uses path concatenation to ensure the base directory is overwritten by an absolute output directory.