Skip to content
This repository has been archived by the owner on Aug 31, 2023. It is now read-only.

Bug Inline comment spacing #2814

Closed
Tracked by #2403 ...
ematipico opened this issue Jul 4, 2022 · 1 comment · Fixed by #3227
Closed
Tracked by #2403 ...

Bug Inline comment spacing #2814

ematipico opened this issue Jul 4, 2022 · 1 comment · Fixed by #3227
Assignees
Labels
A-Formatter Area: formatter S-Bug: confirmed Status: report has been confirmed as a valid bug
Milestone

Comments

@ematipico
Copy link
Contributor

ematipico commented Jul 4, 2022

The following snippet is incorrectly printed:

// Before
test.expect(t => {
	t.true(a);
}, false /*comment */)

// After

test.expect ((t) => {
	t.true(a);
}, false /* comment */ );

Found in #2812

@ematipico ematipico added S-Bug: confirmed Status: report has been confirmed as a valid bug A-Formatter Area: formatter labels Jul 4, 2022
@MichaReiser MichaReiser self-assigned this Jul 18, 2022
@MichaReiser MichaReiser changed the title Bug formatting comments Inline comment spacing Jul 18, 2022
@MichaReiser MichaReiser changed the title Inline comment spacing Bug Inline comment spacing Jul 18, 2022
@github-actions
Copy link

This issue is stale because it has been open 14 days with no activity.

@ematipico ematipico removed the S-Stale label Sep 12, 2022
@MichaReiser MichaReiser linked a pull request Sep 15, 2022 that will close this issue
@MichaReiser MichaReiser added this to the 0.10.0 milestone Sep 22, 2022
@MichaReiser MichaReiser moved this to In Progress in Rome 2022 Sep 22, 2022
Repository owner moved this from In Progress to Done in Rome 2022 Sep 22, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A-Formatter Area: formatter S-Bug: confirmed Status: report has been confirmed as a valid bug
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants