Skip to content

Bump verifai dependency; drop pytest upper bound#466

Merged
dfremont merged 1 commit intomainfrom
dfremont-v3.1.1
May 8, 2026
Merged

Bump verifai dependency; drop pytest upper bound#466
dfremont merged 1 commit intomainfrom
dfremont-v3.1.1

Conversation

@dfremont
Copy link
Copy Markdown
Collaborator

@dfremont dfremont commented May 8, 2026

I ran CI manually now that the new VerifAI release is out, and everything passed. So this PR bumps the requirement so that we disallow old versions of VerifAI which would cause both pygame and pygame-ce to get installed.

Also, in my own testing I noticed that pytest 9 works fine with our hooks, so I've removed the pytest upper bound: if a new version breaks something we should notice in CI and can then fix it (it's only a dev dependency anyway).

@dfremont dfremont requested a review from Eric-Vin May 8, 2026 01:33
@codecov
Copy link
Copy Markdown

codecov Bot commented May 8, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.76%. Comparing base (a8f3bce) to head (f31c8ce).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #466      +/-   ##
==========================================
+ Coverage   89.73%   89.76%   +0.03%     
==========================================
  Files          48       48              
  Lines       13226    13226              
==========================================
+ Hits        11868    11872       +4     
+ Misses       1358     1354       -4     

see 4 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@dfremont dfremont merged commit bdd557c into main May 8, 2026
39 checks passed
@dfremont dfremont deleted the dfremont-v3.1.1 branch May 8, 2026 16:20
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