Skip to content

mformat: force multiline arguments with comma#14861

Merged
eli-schwartz merged 1 commit intomesonbuild:masterfrom
bruchar1:mformat-trailling-comma
Aug 26, 2025
Merged

mformat: force multiline arguments with comma#14861
eli-schwartz merged 1 commit intomesonbuild:masterfrom
bruchar1:mformat-trailling-comma

Conversation

@bruchar1
Copy link
Copy Markdown
Member

@bruchar1 bruchar1 commented Aug 1, 2025

Force multiline arguments when there is a trailing comma. This is the behavior of muon.

Fixes #14721.

Force multiline arguments when there is a trailing comma.
This is the behavior of muon.

Fixes mesonbuild#14721.
@dcbaker dcbaker added this to the 1.10 milestone Aug 1, 2025
@dcbaker
Copy link
Copy Markdown
Member

dcbaker commented Aug 1, 2025

I've added this to the 1.10 milestone, but we could pull it in sooner I guess. Not sure whether we would consider this a bug fix or not?

@Ericson2314
Copy link
Copy Markdown
Contributor

As the author of that issue, thank you!

@Ericson2314
Copy link
Copy Markdown
Contributor

Ericson2314 commented Aug 6, 2025

@dcbaker FWIW, I would love to merge this ASAP, so we can fetch the patch from something more stable (commit on master) in NIx for our formatting CI.

(We'll fetch from this PR in the meantime, but it will be good to upgrade the URL when we can.)

@eli-schwartz
Copy link
Copy Markdown
Member

1.10 is in development now, time to merge. ;)

@eli-schwartz eli-schwartz merged commit 0188454 into mesonbuild:master Aug 26, 2025
32 checks passed
@nirbheek nirbheek modified the milestones: 1.10, 1.9.2 Dec 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

meson format option to preserve trailing commas new lines

5 participants