Skip to content

Fix pycodestyle CI scope for bundled Zcash sources - #2

Merged
cipig merged 3 commits into
masterfrom
copilot/fix-pycodestyle-failure
Aug 2, 2026
Merged

Fix pycodestyle CI scope for bundled Zcash sources#2
cipig merged 3 commits into
masterfrom
copilot/fix-pycodestyle-failure

Conversation

Copilot AI commented Aug 2, 2026

Copy link
Copy Markdown

The pycodestyle workflow started failing because it linted the bundled src/electrumx/lib/zcash subtree, which carries extensive upstream formatting that does not match this repo's lint policy. This change narrows the CI check to repo-owned code and fixes the remaining style violations in tx.py.

  • CI scope

    • Exclude src/electrumx/lib/zcash from the pycodestyle job.
    • Keep the existing lint policy unchanged for the rest of src/.
  • Local style cleanup

    • Fix the remaining non-Zcash pycodestyle findings in src/electrumx/lib/tx.py:
      • spacing after not
      • inline comment spacing
      • missing blank line before TxPIVX
  • Resulting workflow change

    pycodestyle \
      --exclude=src/electrumx/lib/zcash \
      --max-line-length=140 \
      src

Copilot AI changed the title [WIP] Fix the failing GitHub Actions job pycodestyle Fix pycodestyle CI scope for bundled Zcash sources Aug 2, 2026
Copilot AI requested a review from cipig August 2, 2026 04:58
@cipig
cipig marked this pull request as ready for review August 2, 2026 05:17
@cipig
cipig merged commit 57761fd into master Aug 2, 2026
3 checks passed
@cipig
cipig deleted the copilot/fix-pycodestyle-failure branch August 2, 2026 05:18
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