Skip to content

fix: preserve fn newline metadata after semicolons#112

Merged
mhanberg merged 5 commits intoelixir-tools:mainfrom
mkaput:pr/db35a96-fn-newline-metadata
Mar 4, 2026
Merged

fix: preserve fn newline metadata after semicolons#112
mhanberg merged 5 commits intoelixir-tools:mainfrom
mkaput:pr/db35a96-fn-newline-metadata

Conversation

@mkaput
Copy link
Contributor

@mkaput mkaput commented Feb 17, 2026

Capture newline metadata for anonymous functions when the separator is a semicolon token that carries newline count.

This aligns cases like fn ;\n -> :ok end with Elixir metadata while avoiding incorrect newlines: 0 for same-line semicolons.

Adds deterministic regression assertions in the property-regression test block.

Capture newline metadata for anonymous functions when the separator is a semicolon token that carries newline count.

This aligns cases like `fn ;\n -> :ok end` with Elixir metadata while avoiding incorrect `newlines: 0` for same-line semicolons.

Adds deterministic regression assertions in the property-regression test block.
@mhanberg
Copy link
Contributor

need to rebase on main to solve conflicts with other prs

@mhanberg mhanberg force-pushed the pr/db35a96-fn-newline-metadata branch from 9ed39c0 to c410872 Compare March 4, 2026 02:57
@mhanberg mhanberg merged commit 52099ff into elixir-tools:main Mar 4, 2026
35 of 37 checks passed
mhanberg pushed a commit that referenced this pull request Mar 4, 2026
🤖 I have created a release *beep* *boop*
---


##
[0.3.8](v0.3.7...v0.3.8)
(2026-03-04)


### Bug Fixes

* handle ellipsis before <-, ::, and when
([#114](#114))
([221fe49](221fe49))
* parse newline ternary continuation after ellipsis
([#115](#115))
([708da64](708da64))
* preserve fn newline metadata after semicolons
([#112](#112))
([52099ff](52099ff))
* preserve leading nil in stab clause body
([#120](#120))
([f0a79a0](f0a79a0)),
closes [#113](#113)

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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