Skip to content

fix(ci): add ty check to parser folder#8037

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

fix(ci): add ty check to parser folder#8037
leandrodamascena merged 1 commit intodevelopfrom
ci/add-ty-parser

Conversation

@leandrodamascena
Copy link
Contributor

Issue number: closes #7931

Summary

Changes

Resolve all ty diagnostics in utilities/parser/ 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:31
@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/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Mar 16, 2026
@sonarqubecloud
Copy link

@leandrodamascena leandrodamascena requested review from hjgraca and removed request for hjgraca March 16, 2026 11:33
@codecov
Copy link

codecov bot commented Mar 16, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.65%. Comparing base (04436bd) to head (105967f).
⚠️ Report is 1 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #8037   +/-   ##
========================================
  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 merged commit 62591e7 into develop Mar 16, 2026
17 checks passed
@leandrodamascena leandrodamascena deleted the ci/add-ty-parser branch March 16, 2026 11:38
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/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Maintenance: Resolve Ty diagnostics for utilities/parser

2 participants