Skip to content

feat: allow breaking imports/exports when more than one #617

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.

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

Merged
merged 3 commits into from
Apr 2, 2024

Conversation

todor-a
Copy link
Contributor

@todor-a todor-a commented Mar 29, 2024

This extends the work from #473 and allows the option to force break imports/exports when importing/exporting multiple items.

I've changed the forceMultiLineSpecifiers specifier to support one of three values - never | always | onlyWhenMultiple.

@CLAassistant
Copy link

CLAassistant commented Mar 29, 2024

CLA assistant check
All committers have signed the CLA.

@todor-a todor-a changed the title allow breaking imports/exports when more than one [wip] feat: allow breaking imports/exports when more than one Mar 29, 2024
@todor-a todor-a force-pushed the break-on-multiple-imports branch from bbf19b6 to 51bac0a Compare April 1, 2024 07:44
@todor-a todor-a marked this pull request as ready for review April 1, 2024 07:45
@todor-a todor-a changed the title [wip] feat: allow breaking imports/exports when more than one feat: allow breaking imports/exports when more than one Apr 1, 2024
Copy link
Member

@dsherret dsherret left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! Looks good to me. Thanks!

@dsherret dsherret merged commit ae3b724 into dprint:main Apr 2, 2024
@dsherret
Copy link
Member

dsherret commented Apr 2, 2024

@jakebailey FYI - new option for this

@jakebailey
Copy link
Contributor

Wonderful, thank you so much!!!

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.

4 participants