Skip to content

refactor(event_handlers): remove unnecessary init methods#8127

Merged
leandrodamascena merged 1 commit intodevelopfrom
refactor/8122-simplify-resolver-subclasses
Apr 7, 2026
Merged

refactor(event_handlers): remove unnecessary init methods#8127
leandrodamascena merged 1 commit intodevelopfrom
refactor/8122-simplify-resolver-subclasses

Conversation

@leandrodamascena
Copy link
Copy Markdown
Contributor

Issue number: closes #8122

Summary

Refactor init methodos

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 14:02
@leandrodamascena leandrodamascena requested a review from hjgraca April 7, 2026 14:02
@powertools-for-aws-oss-automation powertools-for-aws-oss-automation bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Apr 7, 2026
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 7, 2026

@leandrodamascena leandrodamascena requested a review from svozza April 7, 2026 14:05
@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 (8b42829) to head (2d3442b).
⚠️ Report is 1 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #8127      +/-   ##
===========================================
- Coverage    96.64%   96.64%   -0.01%     
===========================================
  Files          284      284              
  Lines        14082    14079       -3     
  Branches      1153     1153              
===========================================
- Hits         13609    13606       -3     
  Misses         344      344              
  Partials       129      129              

☔ 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 12aeb50 into develop Apr 7, 2026
18 checks passed
@leandrodamascena leandrodamascena deleted the refactor/8122-simplify-resolver-subclasses branch April 7, 2026 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

event_handlers size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Tech debt: Simplify resolver subclasses

2 participants