Skip to content

TST: Set subprocess timeouts consistently on CI - #32198

Open
QuLogic wants to merge 1 commit into
matplotlib:mainfrom
QuLogic:ci-timeout
Open

TST: Set subprocess timeouts consistently on CI#32198
QuLogic wants to merge 1 commit into
matplotlib:mainfrom
QuLogic:ci-timeout

Conversation

@QuLogic

@QuLogic QuLogic commented Aug 10, 2026

Copy link
Copy Markdown
Member

PR summary

Some places do, and some places don't, increase the timeout when running on CI. Instead of having that accidental inconsistency, increase the timeout for everything in the helper function.

For example, I've just now seen test_dpi_change_triggers_resize fail, and it has a fixed timeout of 60 instead of raising it to 120 like other tests.

AI Disclosure

None

PR quality check

  • Use an expressive title, e.g. "Fix title font property precedence"
  • New and changed code is tested
  • [n/a] Plotting related features are demonstrated in an example
  • [n/a] New features and API changes have release notes
  • [n/a] Documentation complies with general and docstring guidelines

Some places do, and some places don't, increase the timeout when running
on CI. Instead of having that accidental inconsistency, increase the
timeout for everything in the helper function.
@QuLogic QuLogic added this to the v3.11.2 milestone Aug 10, 2026
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.

1 participant