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

Implement data.yaml module #4

Merged
merged 99 commits into from
Aug 2, 2024
Merged

Implement data.yaml module #4

merged 99 commits into from
Aug 2, 2024

Conversation

LakshanWeerasinghe
Copy link
Contributor

Purpose

This PR contains all the functionalities that we are going to provide with data.yaml ballerina library. Key features of this Library are,

  • parseString, parseBytes and parseStream apis users can use to create Ballerina values with or without Projection.
  • parsing support for both yaml documents and yaml stream of documents.
  • Support for Fail Safe, JSON, Core Schema parsing using options.
  • toYamlString library to convert a Ballerina value to a yaml string.
  • Compile time expected type checking for parse apis.

Fixes: library-issue-6089

Checklist

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

@LakshanWeerasinghe
Add data.yaml initial implementation
56b7c0d
@LakshanWeerasinghe
Impl yaml lexer
16e8ba4
@LakshanWeerasinghe
Impl yaml parser
cf4eddb
@LakshanWeerasinghe
Add tests for basic types
7a49666
@LakshanWeerasinghe
Fix issue with apply projection
5e05d50
@LakshanWeerasinghe
[Automated] Update the native jar versions
bef0b7d
@LakshanWeerasinghe
Add tests for yaml document parsing
7ade15b
@LakshanWeerasinghe
Implement toYamlString api
c46037c
@LakshanWeerasinghe
Handle union type as expected type
d2d80a9
@LakshanWeerasinghe
Add JsonTraverse to create values from json
4c6914c
@LakshanWeerasinghe
Add tests for union type as expected type
858a6a6
@LakshanWeerasinghe
Support yaml input as byte[] stream
92c443a
@LakshanWeerasinghe
Handle runtime errors as BErrors
216068f
@LakshanWeerasinghe
Add negative tests
fb20398
@LakshanWeerasinghe
Merge branch 'to-yaml-string' into yaml-data-impl
0a30bde
@LakshanWeerasinghe
Add options for parsing
30911de
@LakshanWeerasinghe
Add test after disabling projection
9d9fac6
@LakshanWeerasinghe
Support anchors for maps and sequences
db5d271
@LakshanWeerasinghe
Add tests with anchors
9ee33b9
@LakshanWeerasinghe
Add tag resolution for json and core schemas
cc72c28
@LakshanWeerasinghe
Add tests for tag resolution
bde792e
@LakshanWeerasinghe
Add tests for escaped characters
d0cf4cb
@LakshanWeerasinghe
Fix issue with handling directives
bac9a4c
@LakshanWeerasinghe
Support intersection types and yaml stream parsing
15aa7da
@LakshanWeerasinghe
Impl union and intersection exp type yaml streams
816eaa8
@LakshanWeerasinghe
Fix issue with handling indents
2c32d86
@LakshanWeerasinghe
Fix issue handling espcaed white spaces
2641189
@LakshanWeerasinghe
Add tests for parsing yaml streams
e78c360
@LakshanWeerasinghe
Add tests for toYamlString
a3dac22
@LakshanWeerasinghe
[Automated] Update the native jar versions
1e2b0dc
prakanth97
prakanth97 reviewed Jul 31, 2024
View reviewed changes
ballerina/yaml_api.bal Outdated Show resolved Hide resolved
prakanth97
prakanth97 reviewed Jul 31, 2024
View reviewed changes
ballerina/yaml_api.bal Outdated Show resolved Hide resolved
prakanth97
prakanth97 reviewed Jul 31, 2024
View reviewed changes
ballerina/yaml_api.bal Outdated Show resolved Hide resolved
prakanth97
prakanth97 reviewed Jul 31, 2024
View reviewed changes
native/build.gradle Outdated Show resolved Hide resolved
@prakanth97 prakanth97 requested a review from MaryamZi July 31, 2024 06:50
prakanth97
prakanth97 reviewed Jul 31, 2024
View reviewed changes
README.md Show resolved Hide resolved
prakanth97
prakanth97 previously approved these changes Jul 31, 2024
View reviewed changes
Copy link

@prakanth97 prakanth97 left a comment

Choose a reason for hiding this comment

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

LGTM

SasinduDilshara
SasinduDilshara previously approved these changes Aug 1, 2024
View reviewed changes
prakanth97
prakanth97 previously approved these changes Aug 1, 2024
View reviewed changes
dulajdilshan
dulajdilshan approved these changes Aug 2, 2024
View reviewed changes
Copy link

@dulajdilshan dulajdilshan left a comment

Choose a reason for hiding this comment

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

LGTM!

@LakshanWeerasinghe LakshanWeerasinghe merged commit 0659427 into main Aug 2, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@prakanth97 prakanth97 prakanth97 approved these changes

@dulajdilshan dulajdilshan dulajdilshan approved these changes

@hasithaa hasithaa Awaiting requested review from hasithaa

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

@SasinduDilshara SasinduDilshara Awaiting requested review from SasinduDilshara

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

Successfully merging this pull request may close these issues.

Implement data.yaml module
4 participants
@LakshanWeerasinghe @SasinduDilshara @dulajdilshan @prakanth97