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

Add enum and finite type support for 1.0.x #58

Merged
merged 9 commits into from
Nov 29, 2024

Conversation

SasinduDilshara
Copy link
Contributor

Implement the support for enums and singleton support.

Fixes: ballerina-platform/ballerina-library#7355

@Ajai-Suvendran
Copy link

I verified the fix implemented in this PR by testing it locally, and it resolved the issue as expected.

@@ -1,7 +1,7 @@
[package]
org = "ballerina"
name = "data.xmldata"
version = "1.0.0"
version = "1.0.1"
Copy link
Member

Choose a reason for hiding this comment

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

I think this should be a minor, since the runtime behaviour changes (despite it being a bug previously).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The runtime behaviour only changes for the Finite types ryt?
And also its a error become a valid output.
So no breaking change.

Isnt is better to release a patch?
And also the next minor version will be pack with U11

Copy link
Member

Choose a reason for hiding this comment

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

If someone tested the result to see if it is error, what previously returned true now returns false (or vice versa). While it is an edge case and relies on a bug, it can still be considered breaking.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed in e2ffd01

@gimantha gimantha merged commit f181f9e into ballerina-platform:1.0.x Nov 29, 2024
5 checks passed
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