Skip to content

Fix lint failure from extra blank line in unify tests - #3

Merged
sonnyt merged 2 commits into
mainfrom
copilot/fix-lint-job-failure
Aug 25, 2026
Merged

Fix lint failure from extra blank line in unify tests#3
sonnyt merged 2 commits into
mainfrom
copilot/fix-lint-job-failure

Conversation

Copilot AI commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

The lint GitHub Actions job was failing on a flake8 formatting error in tests/test_unify.py. This change removes the stray spacing that caused the job to fail without altering test behavior.

  • Root cause

    • flake8 reported E303 too many blank lines (2) between adjacent test methods in TestUnifyInitialization
  • Change

    • Remove the extra blank line separating test_drive_has_correct_params and test_me_has_correct_params
  • Impact

    • Restores parity with the repository’s lint rules
    • Keeps the change scoped to test file formatting only
assert unify.drive._connection_id == connection_id

def test_me_has_correct_params(self, api_key, connection_id):
    ...

Co-authored-by: sonnyt <183387+sonnyt@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix failing GitHub Actions job 'lint' Fix lint failure from extra blank line in unify tests Aug 25, 2026
Copilot AI requested a review from sonnyt August 25, 2026 23:21
@sonnyt
sonnyt marked this pull request as ready for review August 25, 2026 23:27
@sonnyt
sonnyt merged commit 3d0e0b5 into main Aug 25, 2026
7 checks passed
@sonnyt
sonnyt deleted the copilot/fix-lint-job-failure branch August 25, 2026 23:27
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