Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve code quality #129

Merged
merged 4 commits into from
Mar 8, 2024
Merged

Improve code quality #129

merged 4 commits into from
Mar 8, 2024

Conversation

tb1337
Copy link
Owner

@tb1337 tb1337 commented Mar 8, 2024

Remove noqa flags and refactor code, fix security issues and code analysis alerts.

@tb1337 tb1337 self-assigned this Mar 8, 2024
@tb1337 tb1337 added refactor Improvement of existing code, not introducing new features. security Marks a security issue that needs to be resolved asap. labels Mar 8, 2024
Copy link

sonarqubecloud bot commented Mar 8, 2024

Copy link

codecov bot commented Mar 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.14%. Comparing base (b054b23) to head (ecceb0f).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #129      +/-   ##
==========================================
- Coverage   98.16%   98.14%   -0.02%     
==========================================
  Files          34       34              
  Lines        1470     1455      -15     
  Branches      191      190       -1     
==========================================
- Hits         1443     1428      -15     
  Misses         13       13              
  Partials       14       14              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@tb1337 tb1337 merged commit 1d69a9f into main Mar 8, 2024
15 of 16 checks passed
@tb1337 tb1337 deleted the code-ql branch March 8, 2024 12:58
@tb1337 tb1337 linked an issue Mar 8, 2024 that may be closed by this pull request
1 task
@github-actions github-actions bot locked and limited conversation to collaborators Mar 10, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
refactor Improvement of existing code, not introducing new features. security Marks a security issue that needs to be resolved asap.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix code scanning alert
1 participant