dart-lang / code_builder

A fluent API for generating valid Dart source code
https://pub.dev/packages/code_builder
BSD 3-Clause "New" or "Revised" License
423 stars 66 forks source link

Add support for extension types #437

Closed srujzs closed 7 months ago

srujzs commented 7 months ago

Corresponds to the syntax defined in https://github.com/dart-lang/language/blob/main/accepted/future-releases/extension-types/feature-specification.md#syntax.

jakemac53 commented 7 months ago

@srujzs fwiw force pushing in GitHub doesn't work well because it destroys all history (I can't tell what you actually changed in that last push). If you can do a merge from master instead in the future that will help me out :).

srujzs commented 7 months ago

fwiw force pushing in GitHub doesn't work well because it destroys all history (I can't tell what you actually changed in that last push).

Ah sorry about that! I've been pushing to avoid unnecessary commits from polluting the branch but messing with the review is way worse.

jakemac53 commented 7 months ago

Ah sorry about that! I've been pushing to avoid unnecessary commits from polluting the branch but messing with the review is way worse.

No worries, really it is just specific to GitHub and how its reviews work. Fwiw when we merge we squash/merge so its always just one commit. So you don't have to worry about branch history really (and it is useful for reviewing so you can see diffs since last review).