Skip to content

Add compliant event handler as extra, for use in other languages#536

Merged
biojppm merged 17 commits intomasterfrom
int_handler
Sep 29, 2025
Merged

Add compliant event handler as extra, for use in other languages#536
biojppm merged 17 commits intomasterfrom
int_handler

Conversation

@biojppm
Copy link
Copy Markdown
Owner

@biojppm biojppm commented Aug 24, 2025

No description provided.

@biojppm biojppm marked this pull request as draft August 24, 2025 16:48
@codecov
Copy link
Copy Markdown

codecov Bot commented Aug 24, 2025

Codecov Report

❌ Patch coverage is 99.44196% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 97.50%. Comparing base (9a044c8) to head (9a52890).
⚠️ Report is 50 commits behind head on master.

Files with missing lines Patch % Lines
src/c4/yml/parse_engine.def.hpp 85.71% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #536      +/-   ##
==========================================
+ Coverage   97.43%   97.50%   +0.06%     
==========================================
  Files          34       44      +10     
  Lines       11600    12835    +1235     
==========================================
+ Hits        11303    12515    +1212     
- Misses        297      320      +23     

☔ 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.

@coveralls
Copy link
Copy Markdown

coveralls commented Aug 24, 2025

Pull Request Test Coverage Report for Build 18081027397

Details

  • 982 of 989 (99.29%) changed or added relevant lines in 17 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.3%) to 97.64%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src_extra/c4/yml/extra/ints_to_testsuite.hpp 11 12 91.67%
tools/yaml_events.cpp 196 197 99.49%
src/c4/yml/parse_engine.def.hpp 30 35 85.71%
Totals Coverage Status
Change from base Build 17971675012: 0.3%
Covered Lines: 13031
Relevant Lines: 13346

💛 - Coveralls

@biojppm biojppm force-pushed the int_handler branch 7 times, most recently from b20d920 to 29ce27e Compare August 31, 2025 10:58
@biojppm biojppm force-pushed the int_handler branch 5 times, most recently from e4deb0f to 50792fa Compare September 18, 2025 17:05
@biojppm biojppm force-pushed the int_handler branch 4 times, most recently from 66a33ee to 42871d4 Compare September 27, 2025 16:29
@biojppm biojppm force-pushed the int_handler branch 3 times, most recently from 161687e to c015e28 Compare September 28, 2025 21:11
@biojppm biojppm marked this pull request as ready for review September 29, 2025 19:15
@biojppm biojppm changed the title Int handler Add compliant event handler as extra, for use in other languages Sep 29, 2025
@biojppm biojppm merged commit f7970d5 into master Sep 29, 2025
302 of 318 checks passed
@biojppm biojppm deleted the int_handler branch September 29, 2025 22:46
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.

2 participants