Skip to content

feat: Implement Display for JsonType - #242

Open
Kriskras99 wants to merge 1 commit into
cloudwego:mainfrom
Kriskras99:main
Open

Kriskras99 wants to merge 1 commit into
cloudwego:mainfrom
Kriskras99:main

Conversation

@Kriskras99

Copy link
Copy Markdown

What type of PR is this?

feat: A new feature

More detailed description for this PR.

en: This makes it easier to write an error message where a field/item has the wrong type.

This makes it easier to write an error message where a field/item
has the wrong type.
@codecov

codecov Bot commented Sep 9, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 71.09%. Comparing base (03545a9) to head (8df6594).

Files with missing lines Patch % Lines
src/value/value_trait.rs 0.00% 9 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #242      +/-   ##
==========================================
- Coverage   71.16%   71.09%   -0.07%     
==========================================
  Files          42       42              
  Lines        9762     9771       +9     
==========================================
  Hits         6947     6947              
- Misses       2815     2824       +9     

☔ View full report in Codecov by Harness.
📢 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.

@Kriskras99

Copy link
Copy Markdown
Author

Formatting errors are not from this PR. I can format them in this PR or provide a separate PR to fix the issues on main.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant