Skip to content

chore: lint hygiene on tests/integration folder#124

Open
qlrd wants to merge 1 commit into
diybitcoinhardware:masterfrom
qlrd:chore/lint-integration-tests
Open

chore: lint hygiene on tests/integration folder#124
qlrd wants to merge 1 commit into
diybitcoinhardware:masterfrom
qlrd:chore/lint-integration-tests

Conversation

@qlrd
Copy link
Copy Markdown
Contributor

@qlrd qlrd commented May 14, 2026

This PR apply some formatting, lint and import sorting on tests/integration folder with ruff and isort (with black profile) tools.

Follow up of #118.

This commit apply some formatting, lint and import sorting on `tests/`
folder with ruff and isort (with black profile) tools.

Follow up of diybitcoinhardware#118.
Copilot AI review requested due to automatic review settings May 14, 2026 15:30
Copy link
Copy Markdown

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.

Pull request overview

Lint and import hygiene applied to the tests/integration folder using ruff and isort (black profile). Imports are reordered into standard groups (stdlib, third-party, local) and split onto individual lines.

Changes:

  • Split multi-import lines into one import per line.
  • Reordered imports per isort/black conventions across test and util modules.
  • No functional code changes.

Reviewed changes

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

Show a summary per file
File Description
tests/integration/run_tests.py Reorder stdlib imports above local util imports.
tests/integration/tests/test_psbt.py Sort imports into stdlib / local / embit groups.
tests/integration/tests/test_pset.py Sort imports into stdlib / local / embit groups.
tests/integration/util/bitcoin.py Alphabetize stdlib imports.
tests/integration/util/liquid.py Add blank line between stdlib and local imports.
tests/integration/util/rpc.py Split combined imports and sort.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@qlrd qlrd changed the title chore: lint hygiene on tests/ folder chore: lint hygiene on tests/integration folder May 14, 2026
@qlrd qlrd marked this pull request as ready for review May 15, 2026 19:20
@qlrd qlrd requested a review from miketlk as a code owner May 15, 2026 19:20
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.

3 participants