Skip to content

refactor(event_handler): refactoring proxy events#8125

Merged
leandrodamascena merged 3 commits intodevelopfrom
refactor/8120-dispatch-dict-proxy-event
Apr 7, 2026
Merged

refactor(event_handler): refactoring proxy events#8125
leandrodamascena merged 3 commits intodevelopfrom
refactor/8120-dispatch-dict-proxy-event

Conversation

@leandrodamascena
Copy link
Copy Markdown
Contributor

Issue number: closes #8120

Summary

Refactor code

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.

@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
svozza
svozza previously approved these changes 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
@leandrodamascena
Copy link
Copy Markdown
Contributor Author

@svozza I need an approval again here, pls! :D

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 7, 2026

@leandrodamascena leandrodamascena requested a review from svozza April 7, 2026 13:01
@svozza
Copy link
Copy Markdown

svozza commented Apr 7, 2026

@svozza I need an approval again here, pls! :D

Done!

@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.62%. Comparing base (c47ab63) to head (5c3b0ac).
⚠️ Report is 1 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #8125      +/-   ##
===========================================
- Coverage    96.63%   96.62%   -0.01%     
===========================================
  Files          284      284              
  Lines        14103    14087      -16     
  Branches      1162     1156       -6     
===========================================
- Hits         13628    13612      -16     
  Misses         345      345              
  Partials       130      130              

☔ 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 merged commit baec69b into develop Apr 7, 2026
17 checks passed
@leandrodamascena leandrodamascena deleted the refactor/8120-dispatch-dict-proxy-event branch April 7, 2026 13:07
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: Use dispatch dict for proxy event conversion

2 participants