Skip to content
This repository was archived by the owner on Jan 28, 2024. It is now read-only.

Added support for generating Unions #215

Merged
merged 11 commits into from
May 20, 2021

Conversation

mannprerak2
Copy link
Contributor

Closes dart-lang/native#358

  • Added support for generating unions.
  • Added and updated tests.
  • Update version, changelog, readme

@mannprerak2 mannprerak2 marked this pull request as ready for review May 19, 2021 08:28
@mannprerak2 mannprerak2 requested a review from dcharkes May 19, 2021 08:28
Copy link
Contributor

@dcharkes dcharkes left a comment

Choose a reason for hiding this comment

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

Good stuff! 🚚

Copy link
Contributor

@dcharkes dcharkes left a comment

Choose a reason for hiding this comment

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

CHANGELOG.md?

Version 3.1.0-dev.0? It's backwards compatible except for the version-bump to Dart 2.14 when we flip it back to stable. And we can release 3.0.x bugfixes if we need to in the mean time.

@mannprerak2
Copy link
Contributor Author

Added changelog.

It's backwards compatible except for the version-bump to Dart 2.14

Would we need to publish 4.0.0 for dart 2.14 when it lands in stable? In that case we probably should use 4.0.0-dev.0.

@dcharkes
Copy link
Contributor

Would we need to publish 4.0.0 for dart 2.14 when it lands in stable? In that case we probably should use 4.0.0-dev.0.

No, it would be fine to publish 3.1.0 for dart 2.14 as long as we satisfy the following constraints:

  • We don't change the public Dart API in lib/*.dart in a non-breaking way.
  • We don't change the command line interface of bin/*.dart in a non-breaking way.

Non-breaking with API is defined as: if behavior observably changes. However, with the exception of thrown Errors. So you could be throwing an UnimplementedError in 3.0.0 and have it implemented in 3.1.0.

In the same way, we refuse the generate unions in 3.0.0 (analogous to throwing Errors in the API), and we implement it in 3.1.0.

There is some other things we could think about, see dart-lang/native#361.

@dcharkes dcharkes merged commit cff315b into dart-archive:master May 20, 2021
@mannprerak2 mannprerak2 deleted the union_support branch May 20, 2021 11:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Generating Unions
2 participants