Closed ZeroCool940711 closed 9 months ago
Attention: 1 lines
in your changes are missing coverage. Please review.
Comparison is base (
53bf520
) 98.56% compared to head (98c37d3
) 98.55%.
Files | Patch % | Lines |
---|---|---|
tests/test_classify.py | 98.91% | 1 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
mmm, it seems like both, the Codecov and SonarCloud.io bots have configuration issues, I have to look into that. Codecov doesnt seem to be looking at the right files, it's looking at what is inside the tests
folder instead of what is on the src
folder. The Sonarcloud.io bot on the other side is not showing the same information here as what is shown on the website or even on the GitHub action, which is actually failing for that, since there are a lot of problems I still have to fix. Most of the changes I am doing are based on what SonarCloud.io shows on their website and what I get with their SonarLint VSCode extension, which is really helpful for some stuff that I would have a hard time figuring out on my own or would have to spend a lot of time on it.
Just do a single PR that is ruff format .
Then on later PRs do other stuff. Ruff does everything that black, flynt, isort, and pyupgrade does but is much faster.
Kudos, no new issues were introduced!
0 New issues
0 Security Hotspots
No data about Coverage
No data about Duplication
This PR includes formatting/spacing fixes, I added more tests for some basic functions and converted many different things to Python 3.8+. This PR will take some time to be ready to merge and it will most likely have breaking changes since we are renaming many functions, arguments and variables from using
mixedCase
tosnake_case
. There are as well many changes to the documentation because of the previously mentioned renaming. I also tried to fix as many bugs, vulnerabilities, issues and code smells reported by SonarCloud.io as I could and I plan to continue doing so until the numbers are as close to 0 as possible, should take a week or two based on the current progress.