Skip to content

Introduce ruff execution on the tests folder #2871#2872

Merged
ahopkins merged 6 commits into
sanic-org:mainfrom
iAndriy:ruff-tests
Dec 12, 2023
Merged

Introduce ruff execution on the tests folder #2871#2872
ahopkins merged 6 commits into
sanic-org:mainfrom
iAndriy:ruff-tests

Conversation

@iAndriy

@iAndriy iAndriy commented Dec 8, 2023

Copy link
Copy Markdown
Contributor

No description provided.

@iAndriy iAndriy requested review from a team as code owners December 8, 2023 03:43
Comment thread Makefile
@iAndriy iAndriy force-pushed the ruff-tests branch 2 times, most recently from 040be04 to a366f70 Compare December 10, 2023 19:58
Comment thread .github/workflows/tests.yml Outdated
Comment thread tests/typing/test_typing.py
@codecov

codecov Bot commented Dec 12, 2023

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (78c44ed) 88.254% compared to head (1b85d7d) 88.240%.

Additional details and impacted files
@@              Coverage Diff              @@
##              main     #2872       +/-   ##
=============================================
- Coverage   88.254%   88.240%   -0.014%     
=============================================
  Files           93        93               
  Lines         7305      7305               
  Branches      1254      1254               
=============================================
- Hits          6447      6446        -1     
- Misses         599       600        +1     
  Partials       259       259               

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

@iAndriy

iAndriy commented Dec 12, 2023

Copy link
Copy Markdown
Contributor Author

@ahopkins all comments resolved and thanks for your review.

@ahopkins ahopkins merged commit 6d28dc2 into sanic-org:main Dec 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants