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

Update YAML module APIs as isolated and update variable name #7

Merged
merged 4 commits into from
May 22, 2023

Conversation

madhukaw
Copy link
Contributor

@madhukaw madhukaw commented May 15, 2023

Purpose

Fixes ballerina-platform/ballerina-library#4439

Examples

Checklist

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

@codecov
Copy link

codecov bot commented May 15, 2023

Codecov Report

Patch coverage: 95.00% and project coverage change: -0.01 ⚠️

Comparison is base (2141d0e) 94.14% compared to head (5f316a0) 94.14%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main       #7      +/-   ##
==========================================
- Coverage   94.14%   94.14%   -0.01%     
==========================================
  Files          44       44              
  Lines        2170     2169       -1     
  Branches     1356     1356              
==========================================
- Hits         2043     2042       -1     
  Misses        127      127              
Impacted Files Coverage Δ
ballerina/modules/composer/collection.bal 92.30% <ø> (ø)
ballerina/modules/composer/scalar.bal 94.91% <ø> (ø)
ballerina/modules/composer/state.bal 100.00% <ø> (ø)
ballerina/modules/emitter/mapping.bal 100.00% <ø> (ø)
ballerina/modules/emitter/sequence.bal 100.00% <ø> (ø)
ballerina/modules/lexer/context.bal 93.15% <ø> (ø)
ballerina/modules/lexer/error.bal 100.00% <ø> (ø)
ballerina/modules/lexer/lexer.bal 86.84% <ø> (ø)
ballerina/modules/parser/data.bal 90.65% <ø> (ø)
ballerina/modules/parser/directive.bal 100.00% <ø> (ø)
... and 26 more

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@madhukaw madhukaw changed the title Update YAML module APIs as isolated Update YAML module APIs as isolated and update variable name May 22, 2023
@shafreenAnfar shafreenAnfar merged commit f9c26d5 into ballerina-platform:main May 22, 2023
warunalakshitha pushed a commit to warunalakshitha/module-ballerina-yaml that referenced this pull request Nov 13, 2024
Update write operations related APIs as isolated
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.

Make YAML module APIs isolated
2 participants