Skip to content

fix(ci): add ty check to dataclasses utility#8038

Merged
leandrodamascena merged 1 commit intodevelopfrom
ci/add-ty-dataclass
Mar 16, 2026
Merged

fix(ci): add ty check to dataclasses utility#8038
leandrodamascena merged 1 commit intodevelopfrom
ci/add-ty-dataclass

Conversation

@leandrodamascena
Copy link
Contributor

Issue number: closes #7934

Summary

Changes

Resolve all ty diagnostics in utilities/data_classes/ and remove from ty exclusion list in pyproject.toml.

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 March 16, 2026 11:52
@boring-cyborg boring-cyborg bot added the dependencies Pull requests that update a dependency file label Mar 16, 2026
@pull-request-size pull-request-size bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Mar 16, 2026
@sonarqubecloud
Copy link

@codecov
Copy link

codecov bot commented Mar 16, 2026

Codecov Report

❌ Patch coverage is 92.30769% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 96.65%. Comparing base (62591e7) to head (d449ff4).
⚠️ Report is 1 commits behind head on develop.

Files with missing lines Patch % Lines
...lambda_powertools/utilities/data_classes/common.py 80.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #8038   +/-   ##
========================================
  Coverage    96.65%   96.65%           
========================================
  Files          279      279           
  Lines        13703    13703           
  Branches      1091     1091           
========================================
  Hits         13244    13244           
  Misses         338      338           
  Partials       121      121           

☔ 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 changed the title fix(ci): add ty check to dataclasses folder fix(ci): add ty check to dataclasses utility Mar 16, 2026
@leandrodamascena leandrodamascena merged commit 1047397 into develop Mar 16, 2026
16 of 17 checks passed
@leandrodamascena leandrodamascena deleted the ci/add-ty-dataclass branch March 16, 2026 12:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file 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.

Maintenance: Resolve Ty diagnostics for utilities/data_classes

2 participants