Skip to content

refactor(event_handler): refactoring params to reduce code#8124

Merged
leandrodamascena merged 2 commits intodevelopfrom
chore/consolidate-param-constructors
Apr 7, 2026
Merged

refactor(event_handler): refactoring params to reduce code#8124
leandrodamascena merged 2 commits intodevelopfrom
chore/consolidate-param-constructors

Conversation

@leandrodamascena
Copy link
Copy Markdown
Contributor

Issue number: closes #8118

Summary

Remove unnecessary docstring. It can inherit from the super class.

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 10:18
@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
@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.63%. Comparing base (7af52e4) to head (a315865).
⚠️ Report is 1 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #8124      +/-   ##
===========================================
- Coverage    96.63%   96.63%   -0.01%     
===========================================
  Files          284      284              
  Lines        14105    14103       -2     
  Branches      1162     1162              
===========================================
- Hits         13630    13628       -2     
  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.

@powertools-for-aws-oss-automation powertools-for-aws-oss-automation bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Apr 7, 2026
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 7, 2026

@leandrodamascena leandrodamascena merged commit c47ab63 into develop Apr 7, 2026
15 checks passed
@leandrodamascena leandrodamascena deleted the chore/consolidate-param-constructors branch April 7, 2026 12:45
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: Consolidate parameter class constructors

2 participants