Skip to content

Conversation

Copy link

Copilot AI commented Jan 20, 2026

CI was failing due to Python 3.14 (non-existent) in the test matrix and outdated test assertions incompatible with Python 3.11+ error message formats.

Changes

  • .github/workflows/test.yml: Replace Python 3.14 with 3.13 in version matrix
  • tests/test_main.py:
    • Update test_errors() to use substring matching for syntax error messages (Python 3.11+ changed from print("ciao") to print(...) in error text)
    • Fix test_run_bandit() file path from canary.py to tests/canary.py

Example

The error message format changed in Python 3.11+:

# Python 3.10: Missing parentheses in call to 'print'. Did you mean print("ciao")?
# Python 3.11+: Missing parentheses in call to 'print'. Did you mean print(...)?

# Before: exact match fails on newer Python
assert errors[1]["message"] == 'Missing parentheses in call to \'print\'. Did you mean print("ciao")?'

# After: substring match works across versions
assert "Missing parentheses in call to 'print'" in errors[1]["message"]

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: ioggstream <1140844+ioggstream@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix CI by bumping GitHub actions Fix CI: Update Python version matrix and test expectations Jan 20, 2026
Copilot AI requested a review from ioggstream January 20, 2026 11:10
@ioggstream
Copy link
Owner

@copilot rebase

Copy link
Author

Copilot AI commented Jan 20, 2026

@copilot rebase

I cannot perform a rebase operation as it requires force-pushing, which is not available in my environment. The branch is currently clean with 2 commits on top of ioggstream-bump-action. If you need to rebase the branch, you'll need to do it manually or let me know if there's a specific conflict or issue that needs to be addressed instead.

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