-
Notifications
You must be signed in to change notification settings - Fork 12
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 read functionality with Jsonpath expressions for Ballerina json datatypes #1
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This commit 5a659c3 contains the base for the jsondata library and it is building properly. It would be better, if you add your changes on top of this. WDYT? @SasinduDilshara @hasithaa
native/src/main/java/io/ballerina/xlibb/jsonpath/BJsonMappingProvider.java
Outdated
Show resolved
Hide resolved
native/src/main/java/io/ballerina/xlibb/jsonpath/BJsonMappingProvider.java
Outdated
Show resolved
Hide resolved
native/src/main/java/io/ballerina/xlibb/jsonpath/BJsonMappingProvider.java
Outdated
Show resolved
Hide resolved
native/src/main/java/io/ballerina/xlibb/jsonpath/BJsonMappingProvider.java
Outdated
Show resolved
Hide resolved
native/src/main/java/io/ballerina/xlibb/jsonpath/BJsonMappingProvider.java
Outdated
Show resolved
Hide resolved
native/src/main/java/io/ballerina/xlibb/jsonpath/BJsonMappingProvider.java
Outdated
Show resolved
Hide resolved
native/src/main/java/io/ballerina/xlibb/jsonpath/BJsonMappingProvider.java
Outdated
Show resolved
Hide resolved
native/src/main/java/io/ballerina/xlibb/jsonpath/BJsonMappingProvider.java
Outdated
Show resolved
Hide resolved
native/src/main/java/io/ballerina/xlibb/jsonpath/BJsonMappingProvider.java
Outdated
Show resolved
Hide resolved
native/src/main/java/io/ballerina/xlibb/jsonpath/BJsonMappingProvider.java
Outdated
Show resolved
Hide resolved
native/src/main/java/io/ballerina/xlibb/jsonpath/BJsonMappingProvider.java
Outdated
Show resolved
Hide resolved
native/src/main/java/io/ballerina/xlibb/jsonpath/BJsonProvider.java
Outdated
Show resolved
Hide resolved
native/src/main/java/io/ballerina/xlibb/jsonpath/BJsonProvider.java
Outdated
Show resolved
Hide resolved
native/src/main/java/io/ballerina/xlibb/jsonpath/BJsonProvider.java
Outdated
Show resolved
Hide resolved
native/src/main/java/io/ballerina/xlibb/jsonpath/BJsonProvider.java
Outdated
Show resolved
Hide resolved
Lets add the commit as a PR, then I will rebase it based on that. |
Mark this as a draft since, the PR that contains base files should be send before this |
@SasinduDilshara Still windows GrallVM validation fails. |
Had a chat with @TharmiganK regarding these failures, This is because this GraalVm checks test with Lang-master branch, I was able to produce that locally now. Will fix it |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Purpose
Fixes ballerina-platform/ballerina-library#5938