Skip to content

Conversation

@jenspapenhagen
Copy link
Collaborator

@jenspapenhagen jenspapenhagen commented Dec 3, 2025

Linked Issue

replace DecodeParser and cleanup on DecodeHelper

Closes #

Description

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Refactoring (no functional changes)
  • Performance improvement
  • Test coverage improvement

Changes Made

SPEC Compliance

  • This PR implements/fixes spec compliance
  • Spec section(s) affected:
  • Spec version:

Testing

  • All existing tests pass
  • Added new tests for changes
  • Tests cover edge cases and spec compliance

Pre-submission Checklist

  • My code follows the project's coding standards
  • I have run code formatting/linting tools
  • I have added tests that prove my fix/feature works
  • New and existing tests pass locally
  • I have updated documentation if needed
  • I have reviewed the TOON specification for relevant sections

Breaking Changes

  • No breaking changes
  • Breaking changes (describe migration path below)

Additional Context

@jenspapenhagen jenspapenhagen requested a review from a team as a code owner December 3, 2025 17:46
@jenspapenhagen jenspapenhagen marked this pull request as draft December 3, 2025 19:06
@jenspapenhagen jenspapenhagen marked this pull request as ready for review December 3, 2025 20:46
@felipestanzani felipestanzani merged commit aa4cee2 into toon-format:main Dec 3, 2025
1 check passed
@github-actions
Copy link
Contributor

github-actions bot commented Dec 3, 2025

Code Coverage

Overall Project 98.22% -0.14% 🍏
Files changed 95.68% 🍏

File Coverage
TabularArrayDecoder.java 99.32% 🍏
ObjectDecoder.java 97.97% 🍏
ValueDecoder.java 95.92% -4.08% 🍏

@jenspapenhagen jenspapenhagen deleted the decodeParser branch December 9, 2025 19:36
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