Skip to content

Fix child decl ordering - #2984

Merged
hdgarrood merged 2 commits into
masterfrom
fix-child-decl-ordering
Jul 10, 2017
Merged

Fix child decl ordering#2984
hdgarrood merged 2 commits into
masterfrom
fix-child-decl-ordering

Conversation

@hdgarrood

Copy link
Copy Markdown
Contributor

See commit descriptions.

hdgarrood added 2 commits July 9, 2017 22:40
Child declarations - that is, data constructors, type class members, and
instances - were appearing in the reverse order because of how the
`partitionChildren` function works.

This fixes purescript/pursuit#125.
This doesn't fully test what we want it to, as it is still possible to
accidentally reverse the list during HTML rendering. However that's not
too difficult to verify manually (as I have just done).

@paf31 paf31 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@hdgarrood
hdgarrood merged commit 7fdd39d into master Jul 10, 2017
@hdgarrood
hdgarrood deleted the fix-child-decl-ordering branch July 10, 2017 00:59
paf31 added a commit that referenced this pull request Jul 10, 2017
paf31 added a commit that referenced this pull request Jul 10, 2017
…2985)

* Revert "Fix child decl ordering (#2984)"

This reverts commit 7fdd39d.

* Revert "Only build master and semver tags in Travis (#2982)"

This reverts commit c41a7a4.

* Revert "Include comments in child declarations in HTML docs (#2981)"

This reverts commit ad6166f.

* Revert "Solve RowToList (#2945)"

This reverts commit 0e2d820.

* Revert "Fix links to type operators in HTML docs (#2979)"

This reverts commit 5fe26f2.

* Revert "Re-evaluate REPL globs during :reload (#2978)"

This reverts commit 50e1c68.

* Revert "[purs ide] Collect data constructors (#2976)"

This reverts commit c965f35.

* Revert "Replace synonyms in instance constraints (#2973)"

This reverts commit ee66e0d.

* Revert "Require single-method instance declarations to be indented (#2965)"

This reverts commit b3e470d.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants