Skip to content

chore: enable linting with ruff#42

Merged
rilshok merged 9 commits intodevfrom
rilshok/ruff
Apr 5, 2025
Merged

chore: enable linting with ruff#42
rilshok merged 9 commits intodevfrom
rilshok/ruff

Conversation

@rilshok
Copy link
Owner

@rilshok rilshok commented Apr 5, 2025

No description provided.

@github-actions
Copy link

github-actions bot commented Apr 5, 2025

Coverage

Coverage Report
FileStmtsMissCoverMissing
src/iokit
   checksum.py694929%18, 21, 25–44, 49–69, 76–77, 81–84, 89, 92–94
   state.py1413178%32–33, 52, 57–60, 91–94, 99, 102–103, 110, 118–120, 128, 143–144, 168, 171–172, 177–180, 184, 196–197
src/iokit/extensions
   audio.py76495%82, 103–105, 109
   auto.py56493%75, 86–88
   enc.py87891%91, 112–116, 119–120
   tar.py26292%35, 38
   zip.py25196%32
src/iokit/storage
   local.py1429136%41–42, 44–45, 60–61, 64, 67–71, 74–78, 81, 84–89, 94–95, 98–102, 105–108, 111–114, 117, 120–122, 136–144, 147–155, 158–162, 165–171, 174–179, 182–195, 198–202, 205, 208–209
   storage.py482156%17–18, 22–23, 27–28, 32–33, 37–38, 50, 53, 56–60, 63–64, 67, 70
   web.py24579%14, 20–21, 26–27
TOTAL86821675% 

Tests Skipped Failures Errors Time
52 0 💤 0 ❌ 0 🔥 7.117s ⏱️

@rilshok rilshok requested a review from Copilot April 5, 2025 15:35
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot reviewed 23 out of 23 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (1)

src/iokit/storage/web.py:18

  • Typo in the variable name 'resopnse'; it should be renamed to 'response'.
    resopnse = requests.get(url, timeout=timeout)

@rilshok rilshok requested a review from Copilot April 5, 2025 16:15
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot reviewed 23 out of 23 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (1)

src/iokit/checksum.py:52

  • Consider verifying that 'data.buffer' is indeed an instance of BytesIO, as the removed assertion may lead to issues if it is not.
buffer = data.buffer

@rilshok rilshok merged commit cc476a3 into dev Apr 5, 2025
4 checks passed
@rilshok rilshok deleted the rilshok/ruff branch April 5, 2025 16:16
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