Skip to content

Fix CI build failure by resolving flake8 blockers and stabilizing pytest step - #1

Merged
hakanyorukk merged 2 commits into
mainfrom
copilot/fix-github-actions-build-job
Aug 21, 2026
Merged

hakanyorukk merged 2 commits into
mainfrom
copilot/fix-github-actions-build-job

Conversation

Copilot AI commented Aug 21, 2026 •

Copy link
Copy Markdown
Contributor

The build GitHub Actions job failed on lint-gated errors (E999, F821, F632) in committed Python files. This PR removes those blockers and adds focused regression tests so the workflow’s pytest step no longer fails due to empty test discovery.

  • Lint-blocking syntax/name fixes

    • Corrected invalid f-string quoting in a2/exercise_1.py and a2/exercise_2.py.
    • Fixed undefined function reference in basic_math_recursion/task3/exercise_4.py by calling the existing function.
    • Replaced identity comparison with value comparison in h1/exercise_1.py (is → == for string literal).
  • Targeted regression coverage

    • Added tests/test_build_fixes.py with focused assertions for:
      • status-code bucket aggregation (2xx, 4xx, 5xx) in both a2 exercises.
      • recursive Fibonacci path used by exercise_4.
  • Representative change

    # before
    status = f"{request["status"][0]}xx"
    
    # after
    status = f"{request['status'][0]}xx"

@hakanyorukk
hakanyorukk marked this pull request as ready for review August 21, 2026 09:37
Co-authored-by: hakanyorukk <125490544+hakanyorukk@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix failing GitHub Actions job build Fix CI build failure by resolving flake8 blockers and stabilizing pytest step Aug 21, 2026
Copilot AI requested a review from hakanyorukk August 21, 2026 09:39
@hakanyorukk
hakanyorukk merged commit e8312bc into main Aug 21, 2026
1 of 2 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.

2 participants