Skip to content
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

Remove support for fixes and --fix. #1564

Merged
merged 2 commits into from
Sep 11, 2024
Merged

Remove support for fixes and --fix. #1564

merged 2 commits into from
Sep 11, 2024

Conversation

munificent
Copy link
Member

The tools that come with the Dart SDK provide two ways to apply automated changes to code: dart format --fix and dart fix. The former is older and used to be faster. But it can only apply a few fixes and hasn't been maintained in many years. The dart fix command is actively maintained, can apply all of the fixes that dart format --fix could apply and many many more.

In order to avoid duplicate engineering effort, we decided to consolidate on dart fix as the one way to make automated changes that go beyond the simple formatting and style changes that dart format applies.

The ability to apply fixes is also removed from the DartFormatter() library API.

cc @mit-mit, @keertip

The tools that come with the Dart SDK provide two ways to apply automated changes to code: `dart format --fix` and `dart fix`. The former is older and used to be faster. But it can only apply a few fixes and hasn't been maintained in many years. The `dart fix` command is actively maintained, can apply all of the fixes that `dart format --fix` could apply and many many more.

In order to avoid duplicate engineering effort, we decided to consolidate on `dart fix` as the one way to make automated changes that go beyond the simple formatting and style changes that `dart format` applies.

The ability to apply fixes is also removed from the `DartFormatter()` library API.
Copy link
Member

@natebosch natebosch left a comment

Choose a reason for hiding this comment

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

So much deleted code 🎉

@munificent munificent merged commit 87e527d into main Sep 11, 2024
7 checks passed
@munificent munificent deleted the remove-fix branch September 11, 2024 20:18
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.

3 participants