Skip to content

refactor(event_handler): refactoring encoder file#8126

Merged
leandrodamascena merged 2 commits intodevelopfrom
refactor/8121-simplify-jsonable-encoder
Apr 7, 2026
Merged

refactor(event_handler): refactoring encoder file#8126
leandrodamascena merged 2 commits intodevelopfrom
refactor/8121-simplify-jsonable-encoder

Conversation

@leandrodamascena
Copy link
Copy Markdown
Contributor

Issue number: closes #8121

Summary

Refactor json encoder function.

Changes

Please provide a summary of what's being changed

User experience

Please share what the user experience looks like before and after this change


By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

@leandrodamascena leandrodamascena requested a review from a team as a code owner April 7, 2026 13:36
@leandrodamascena leandrodamascena requested a review from sdangol April 7, 2026 13:36
@powertools-for-aws-oss-automation powertools-for-aws-oss-automation bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Apr 7, 2026
@powertools-for-aws-oss-automation powertools-for-aws-oss-automation bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Apr 7, 2026
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 7, 2026

@leandrodamascena leandrodamascena requested a review from a team April 7, 2026 13:44
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.64%. Comparing base (baec69b) to head (6036aa1).
⚠️ Report is 1 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #8126      +/-   ##
===========================================
+ Coverage    96.62%   96.64%   +0.01%     
===========================================
  Files          284      284              
  Lines        14087    14082       -5     
  Branches      1156     1153       -3     
===========================================
- Hits         13612    13609       -3     
+ Misses         345      344       -1     
+ Partials       130      129       -1     

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

@leandrodamascena leandrodamascena requested a review from svozza April 7, 2026 13:50
@leandrodamascena leandrodamascena merged commit 8b42829 into develop Apr 7, 2026
17 checks passed
@leandrodamascena leandrodamascena deleted the refactor/8121-simplify-jsonable-encoder branch April 7, 2026 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

event_handlers size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Tech debt: Simplify jsonable_encoder

2 participants