Repository navigation
Support primary constructors - #1397
donny-dont wants to merge 1 commit into
Conversation
Change the check for the `_` value constructor to support `new _()`.
|
The check for the factory constructor works with the primary constructor feature of dart 3.13 out of the box so just targeting the |
|
Thanks! I will take a look, probably there is additional work that is useful with this. |
|
@davidmorgan is there anything I can do to move things along? Without this change anyone trying to use built value classes cannot update to the new primary constructor format. The generator will refuse to accept the new syntax as being valid. |
|
@donny-dont commenting here is a good way :) I'll try to get to it this week. Thanks :) |
|
@davidmorgan sounds good! I don't know what Dart SDK's this is tested against but if its |
|
Looks like you've got this covered in #1400 so I'm going to close this and let you do your thing! Thanks for getting this figured out! |
Change the check for the
_value constructor to supportnew _().