Skip to content

fix: remove mixed tabs/spaces indentation in bangbang.py (#376)#434

Merged
pradeeban merged 1 commit into
ControlCore-Project:devfrom
GaneshPatil7517:fix/bangbang-mixed-indentation
Feb 20, 2026
Merged

fix: remove mixed tabs/spaces indentation in bangbang.py (#376)#434
pradeeban merged 1 commit into
ControlCore-Project:devfrom
GaneshPatil7517:fix/bangbang-mixed-indentation

Conversation

@GaneshPatil7517
Copy link
Copy Markdown
Contributor

@GaneshPatil7517 GaneshPatil7517 commented Feb 20, 2026

hey Pradeeban Sir
Fixes #376 Remove mixed tabs and spaces indentation in bangbang.py.

Problem
Both tools/bangbang.py (line 12) and humanc/bangbang.py (line 11) contained
blank lines with mixed tab and space characters (\t \r\n). This violates
PEP 8 indentation rules and can cause TabError in Python 3 when tabs and
spaces are mixed in the same block.

Fix
Replaced the mixed-indentation blank lines with clean empty lines in both files.
No logic changes purely whitespace cleanup.

Files Changed

  • tools/bangbang.py removed tab+spaces on blank line 12
  • humanc/bangbang.py removed tab+spaces on blank line 11

Verification
Confirm no tabs remain

python -c "
for path in ['tools/bangbang.py', 'humanc/bangbang.py']:
    with open(path, 'rb') as f:
        assert b'\t' not in f.read(), f'Tabs in {path}'
    print(f'PASS: {path}')
"

Confirm both files compile
python -m py_compile tools/bangbang.py
python -m py_compile humanc/bangbang.py

Copilot AI review requested due to automatic review settings February 20, 2026 06:01
Copy link
Copy Markdown
Contributor

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

This PR addresses Issue #376 by removing mixed tab/space characters on blank lines in the two bangbang.py controller scripts, preventing potential Python TabError and aligning with PEP 8 indentation expectations.

Changes:

  • Removed tab+spaces characters on an otherwise blank line in tools/bangbang.py.
  • Removed tab+spaces characters on an otherwise blank line in humanc/bangbang.py.

Reviewed changes

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

File Description
tools/bangbang.py Replaces a mixed-indentation blank line with a clean empty line.
humanc/bangbang.py Replaces a mixed-indentation blank line with a clean empty line.

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

Comment thread tools/bangbang.py Outdated
Comment thread humanc/bangbang.py Outdated
@GaneshPatil7517 GaneshPatil7517 force-pushed the fix/bangbang-mixed-indentation branch from 28c0f75 to 93a8759 Compare February 20, 2026 06:06
@pradeeban pradeeban merged commit 97a38f6 into ControlCore-Project:dev Feb 20, 2026
6 checks passed
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