New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add community build (a-la scalameta), fix formatting bugs #4133

Open

kitbellew opened this issue Aug 3, 2024 · 0 comments

Collaborator

kitbellew commented Aug 3, 2024

Test formatting of open-source community code, similar to how it was done in scalameta, and fix any formatting bugs discovered this way.

This was referenced Aug 6, 2024

FormatOps: optional braces if ends in lone comment #4134

Merged

Error: add reason to SearchStateExploded #4135

Merged

Router: no space after => if empty body #4136

Merged

Router: apply src=keep around : return-type #4137

Merged

Router: extend optimal token to include comment #4138

Merged

FormatOps: fix cond between while and do #4139

Merged

Router: disambiguate end marker in keyword rules #4140

Merged

Router: combine two case splits via policies #4141

Merged

RedundantBraces: don't remove if annotated #4143

Merged

TreeOps: handle removed braces with nested lambdas #4144

Merged

FormatOps: Ctrl body can be tuple in removed block #4145

Merged

ScannerTokens: check end marker for try/for/while scalameta/scalameta#3888

Merged

RedundantBraces: don't remove empty blocks #4148

Merged

ScannerTokens: check end marker for if as well scalameta/scalameta#3889

Merged

ScalametaParser: remove EndPosPreOutdent, invalid scalameta/scalameta#3890

Merged

ScalametaParser: terminate generator on outdent scalameta/scalameta#3891

Merged

FormatOps: don't break within comment in ctrl body #4149

Merged

Split: expose mod via modExt in Split/State #4150

Merged

FormatWriter: move parens rewrite checks elsewhere #4151

Merged

FormatWriter: modify brace-to-paren rewrite steps #4152

Merged

Router: no space around { to be changed to ( #4153

Merged

FormatTokens: get last skipping attached comments #4154

Merged

FormatWriter: insert end marker after semicolon #4155

Merged

ScannerTokens: consistently strip regions on EOL scalameta/scalameta#3892

Merged

This was referenced Aug 13, 2024

Router: align costs to ignore current break #4157

Merged

Router: define chain splits as dedicated variables #4158

Merged

Router: use optimal token for brace-ending chains #4159

Merged

FormatOps: find optional braces in nested blocks #4160

Merged

FormatTokens: move getBraces from TreeOps #4161

Merged

FormatOps: definite NL in OptionalBraces.BlockImpl #4162

Merged

This was referenced Sep 26, 2024

FormatOps: add first param to arguments with mod #4334

Merged

Split: separate original cost and later penalty #4335

Merged

BestFirstSearch: updateBest both states #4336

Merged

Router: bp=defnsite ( use first comma as optimal #4337

Merged

BestFirstSearch: optimal has low-penalty NL split #4338

Merged

RedundantBraces: else-if always ok if opt-braces #4339

Merged

CommunitySuite: support excluding some styles #4340

Merged

CommunitySuite: add tests for scala2 versions #4341

Merged

CommunitySuite: add tests for scala3 versions #4342

Merged

CommunitySuite: print git ref and scalafmt style #4344

Merged

CommunityRepoSuite: add common abstract superclass #4345

Merged

CommunityTests: split into separate projects #4346

Merged

CommunitySuite: add tests for scala-cli versions #4347

Merged

Router: extract single stat from rewritten block #4349

Merged

Route: hide NL before dot for keep via a splittag #4351

Merged

RemoveScala3OptionalBraces: not with no-NL select #4352

Merged

FormatOps: extend slb end past select for keep #4353

Merged

BestFirstSearch: arg clause an opt-zone for keep #4354

Merged

Router: fix consistency based on src NL existence #4355

Merged

RedundantBraces: keep around do-while within for #4356

Merged

TreeOps: get owners via token range, not position #4357

Merged

Router: check if "NL" for ( is actually not NL #4359

Merged

CommunitySparkSuite: add more spark versions #4360

Merged

BestFirstSearch: stop using State.allAltAreNL #4369

Open

FormatOps: no blank within infix if not enclosed #4370

Merged

Router: price sole NL split in => <NL> at 1 #4371

Merged

TreeOps: revise NL-optimization around functions #4372

Merged

CommunitySuite: test scala.js version 1.17.0 #4373

Merged

RedundantBraces: keep some around PartialFunction #4374

Merged

TreeOps: ignore single block stat for NL optimizn #4375

Open

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