Skip to content

Comments

Add compiler validation through compiler plugin#3

Merged
niveathika merged 8 commits intoballerina-platform:mainfrom
niveathika:main
May 14, 2025
Merged

Add compiler validation through compiler plugin#3
niveathika merged 8 commits intoballerina-platform:mainfrom
niveathika:main

Conversation

@niveathika
Copy link
Contributor

Purpose

$subject

Fixes:

Examples

Checklist

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

@niveathika niveathika requested a review from daneshk as a code owner May 8, 2025 19:48
@codecov
Copy link

codecov bot commented May 8, 2025

Codecov Report

Attention: Patch coverage is 84.13598% with 56 lines in your changes missing coverage. Please review.

Project coverage is 51.54%. Comparing base (b018d40) to head (cba3de2).
Report is 9 commits behind head on main.

Files with missing lines Patch % Lines
...b/cdc/compiler/validator/CdcFunctionValidator.java 79.35% 14 Missing and 18 partials ⚠️
...cdc/compiler/validator/CdcServiceAnalysisTask.java 74.19% 4 Missing and 4 partials ⚠️
...c/compiler/codeaction/AbstractCdcCodeTemplate.java 82.35% 3 Missing and 3 partials ⚠️
...main/java/io/ballerina/lib/cdc/compiler/Utils.java 72.22% 2 Missing and 3 partials ⚠️
...ib/cdc/compiler/validator/CdcServiceValidator.java 89.58% 2 Missing and 3 partials ⚠️

❌ Your project status has failed because the head coverage (51.54%) is below the target coverage (80.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files
@@              Coverage Diff              @@
##               main       #3       +/-   ##
=============================================
+ Coverage     29.23%   51.54%   +22.31%     
- Complexity       23      138      +115     
=============================================
  Files            15       26       +11     
  Lines           520      875      +355     
  Branches        129      180       +51     
=============================================
+ Hits            152      451      +299     
- Misses          364      389       +25     
- Partials          4       35       +31     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@niveathika niveathika force-pushed the main branch 3 times, most recently from 5330df9 to c16dd76 Compare May 9, 2025 02:07
Copy link
Member

@ThisaruGuruge ThisaruGuruge left a comment

Choose a reason for hiding this comment

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

LGTM

@niveathika niveathika merged commit a2e56a7 into ballerina-platform:main May 14, 2025
3 of 4 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