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.

Sign up for GitHub

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

Jump to bottom

Migrate runtime APIs for Java 21 #53

Merged
merged 45 commits into from
Nov 13, 2024
Merged

Migrate runtime APIs for Java 21 #53

merged 45 commits into from
Nov 13, 2024

Conversation

HindujaB
Copy link
Contributor

Purpose

$subject

Fixes:

Examples

Checklist

  • Linked to an issue
  • Updated the changelog
  • Added tests
  • Updated the spec
  • Checked native-image compatibility

SasinduDilshara and others added 30 commits October 11, 2024 13:44
@SasinduDilshara
[Automated] Update the native jar versions
8e98ad1
@SasinduDilshara
[Automated] Update the native jar versions
a9b93cc
@SasinduDilshara
Fix the invalid error for readonly record types
de25263
@gimantha
Merge branch 'main' into fix_7253
b8c1596
@SasinduDilshara
Added tests for readonly record fields
b540cf8
@SasinduDilshara
Merge branch 'main' of https://github.com/ballerina-platform/module-b…
2dbd84d 
…allerina-data.jsondata into fix_7253
@SasinduDilshara
Merge branch 'fix_7253' of https://github.com/SasinduDilshara/module-…
8fe12eb 
…ballerina-data.jsondata into fix_7253
@SasinduDilshara
Add tests for immutable types
900ac32
@SasinduDilshara
Add tests for readonly fields
3dce590
@SasinduDilshara
Update the codeowner file
9019906
@SasinduDilshara
Add tests for selectively immutable types
28c9eb6
@SasinduDilshara
[Automated] Update the native jar versions
ca87b29
@SasinduDilshara
[Automated] Update the native jar versions
6e63d1d
@SasinduDilshara
[Automated] Update the native jar versions
17493fe
@SasinduDilshara
Add JSON value creation for readonly record values
86bb2f7
@SasinduDilshara
Fix issue in readonly array field types with parseString APIs
c34d096
@SasinduDilshara
Add test for non-readonly record with readonly fields
6ca0ca5
@SasinduDilshara
Add assertions for non-readonly fields in json tests
754d86c
@SasinduDilshara
Add asertions for readonly fields and records
af8b930
@SasinduDilshara
Added tests for selectively immutable fields
43d1475
@SasinduDilshara
Add assertions for readonly fields
8f4c00d
@SasinduDilshara
Merge pull request ballerina-platform#46 from SasinduDilshara/issue_7259
4791e2b 
Add JSON to record conversion for readonly records in `parseAsType` API
@SasinduDilshara
Merge branch 'main' of https://github.com/ballerina-platform/module-b…
8a6d5f6 
…allerina-data.jsondata into fix_7253
@SasinduDilshara
Update get member type function with intersection types
5dcb0ad
@SasinduDilshara
Added tests with refered type for selectively immutable fields
9d9653c
@SasinduDilshara
Merge pull request ballerina-platform#45 from SasinduDilshara/fix_7253
2e473c5 
Fix the invalid error for readonly record types
@SasinduDilshara
Fix issues with map types when data projection is false
61e791c
@SasinduDilshara
Added test for nlable types with map as expected types
3e80cae
@SasinduDilshara
Merge branch 'main' of https://github.com/ballerina-platform/module-b…
ce1b864 
…allerina-data.jsondata into issue_7329
@SasinduDilshara
Update testMapAsExpectedTypeWithJsonSource test function
907da11
MaryamZi and others added 15 commits November 11, 2024 17:35
@MaryamZi
Fix toJson for cyclic values
8647dce
@SasinduDilshara
Merge pull request ballerina-platform#51 from ballerina-platform/fix-…
4156d57 
…7326

Fix `toJson` for cyclic values
@MaryamZi
Fix toJson not using the Name annotation with tables
4bf064c
@SasinduDilshara
Add union tests for parseAsType function with data projection false
aa624d1
@MaryamZi
Add tests
07d366f
@gimantha
Merge branch 'main' into issue_7329
4a15be9
@SasinduDilshara
Merge pull request ballerina-platform#52 from MaryamZi/fix-7348
f38fed0 
Fix the `Name` annotation not being used when using `toJson` with tables
@SasinduDilshara
update testMapAsExpectedTypeWithJsonSource tests with aseert type
8e98e64
@SasinduDilshara
Merge branch 'issue_7329' of https://github.com/SasinduDilshara/modul…
e39f05b 
…e-ballerina-data.jsondata into issue_7329
@SasinduDilshara
Merge branch 'main' of https://github.com/ballerina-platform/module-b…
21e654a 
…allerina-data.jsondata into issue_7329
@SasinduDilshara
Merge pull request ballerina-platform#49 from SasinduDilshara/issue_7329
94297e4 
Fix invalid compile time errors when expected type is map and data projection is false
@HindujaB
Fix merge conflicts
51d846b
@HindujaB
[Automated] Update the native jar versions
d96dbe6
@HindujaB
Migrate runtime APIs
53247da
@HindujaB
Revert "[Automated] Update the native jar versions"
73ca5a3 
This reverts commit d96dbe6.
@codecov Codecov
Copy link

codecov bot commented Nov 13, 2024

Codecov Report

Attention: Patch coverage is 81.81818% with 8 lines in your changes missing coverage. Please review.

Please upload report for BASE (java21@07653f7). Learn more about missing BASE report.

Files with missing lines Patch % Lines
...ballerina/lib/data/jsondata/json/JsonTraverse.java 66.66% 2 Missing and 1 partial ⚠️
...va/io/ballerina/lib/data/jsondata/json/Native.java 88.46% 2 Missing and 1 partial ⚠️
...ata/jsondata/io/BallerinaByteBlockInputStream.java 50.00% 1 Missing ⚠️
...o/ballerina/lib/data/jsondata/json/JsonParser.java 0.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files 
@@            Coverage Diff            @@
##             java21      #53   +/-   ##
=========================================
  Coverage          ?   81.27%           
  Complexity        ?      323           
=========================================
  Files             ?       25           
  Lines             ?     1768           
  Branches          ?      338           
=========================================
  Hits              ?     1437           
  Misses            ?      202           
  Partials          ?      129

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@warunalakshitha warunalakshitha merged commit 6d332e1 into ballerina-platform:java21 Nov 13, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@warunalakshitha warunalakshitha warunalakshitha approved these changes

@gimantha gimantha Awaiting requested review from gimantha gimantha is a code owner

@MaryamZi MaryamZi Awaiting requested review from MaryamZi MaryamZi is a code owner

@hasithaa hasithaa Awaiting requested review from hasithaa hasithaa is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@HindujaB @warunalakshitha @SasinduDilshara @gimantha @MaryamZi