Skip to content

Add attached derive clause formatting#134

Draft
i-am-the-slime wants to merge 2 commits into
natefaubion:mainfrom
i-am-the-slime:attached-derive-clauses
Draft

Add attached derive clause formatting#134
i-am-the-slime wants to merge 2 commits into
natefaubion:mainfrom
i-am-the-slime:attached-derive-clauses

Conversation

@i-am-the-slime

@i-am-the-slime i-am-the-slime commented Apr 3, 2026

Copy link
Copy Markdown
Contributor

Warning

This PR was AI-generated. If that's not okay close it! I'm just trying to make PureScript even nicer

Summary

  • Adds formatting for derive (Class), derive newtype (Class), and derive (Class) via Type clauses
  • Derive clauses are indented under their parent data/newtype declaration
  • Uses Grouped layout so class lists stay on one line when they fit
  • All existing tests pass, new snapshot test added

Depends on natefaubion/purescript-language-cst-parser (attached-derive-clauses branch).
Companion to purescript/purescript#4592

@i-am-the-slime

Copy link
Copy Markdown
Contributor Author

CI fails because it pulls language-cst-parser from the registry, which doesn't have the new types yet. This will pass once natefaubion/purescript-language-cst-parser#66 is merged and released.

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.

1 participant