8000 Increase timeout for interactive backend tests by QuLogic · Pull Request #25295 · matplotlib/matplotlib · GitHub
[go: up one dir, main page]

Skip to content

Conversation

@QuLogic
Copy link
Member
@QuLogic QuLogic commented Feb 22, 2023

PR Summary

It seems that macOS is failing a bit more now; hopefully a slightly longer timeout is all that is needed.

PR Checklist

Documentation and Tests

  • Has pytest style unit tests (and pytest passes)
  • [n/a] Documentation is sphinx and numpydoc compliant (the docs should build without error).
  • [n/a] New plotting related features are documented with examples.

Release Notes

  • [n/a] New features are marked with a .. versionadded:: directive in the docstring and documented in doc/users/next_whats_new/
  • [n/a] API changes are marked with a .. versionchanged:: directive in the docstring and documented in doc/api/next_api_changes/
  • [n/a] Release notes conform with instructions in next_whats_new/README.rst or next_api_changes/README.rst

It seems that macOS is failing a bit more now; hopefully a slightly
longer timeout is all that is needed.
@QuLogic QuLogic added topic: testing CI: testing CI configuration and testing labels Feb 22, 2023
@QuLogic QuLogic added this to the v3.7.1 milestone Feb 22, 2023
@QuLogic QuLogic merged commit 948f233 into matplotlib:main Feb 23, 2023
@QuLogic QuLogic deleted the longer-test-timeout branch February 23, 2023 03:05
meeseeksmachine pushed a commit to meeseeksmachine/matplotlib that referenced this pull request Feb 23, 2023
rcomer added a commit that referenced this pull request Feb 23, 2023
…295-on-v3.7.x

Backport PR #25295 on branch v3.7.x (Increase timeout for interactive backend tests)
@QuLogic
Copy link
Member Author
QuLogic commented Mar 2, 2023

Sadly, it appears this did not fix anything https://github.com/matplotlib/matplotlib/actions/runs/4301642051/jobs/7499122464#step:13:468 Maybe it's just really slow or there's something else going on.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI: testing CI configuration and testing topic: testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

0