CI fixes because other packages break other packages

[skrawcz]

Fixes for CI.

Contains 1 temporary fix around plotly.

Changes

How I tested this

Notes

Checklist

• PR has an informative and human-readable title (this will be pulled into the release notes)
• Changes are limited to a single goal (no scope creep)
• Code passed the pre-commit check & code is left cleaner/nicer than when first encountered.
• Any change in functionality is tested
• New functions are documented (with a description, list of inputs, and expected output)
• Placeholder code is flagged / future TODOs are captured in comments
• Project documentation has been updated if adding/changing functionality.

[ellipsis-dev]

❌ Changes requested. Reviewed everything up to afd93d3 in 42 seconds

More details

• Looked at 61 lines of code in 2 files
• Skipped 0 files when reviewing.
• Skipped posting 1 drafted comments based on config settings.

1. .ci/test.sh:31

• Draft comment:
  Consider using a more explicit version check for clarity:

    if python -c 'import sys; exit(0) if sys.version_info >= (3, 9) else exit(1)'; then

• Reason this comment was not posted:
  Confidence changes required: 50%
  The PR adds a version check for Python 3.9+ in the test script and test cases, which is a good practice to ensure compatibility. However, the version check in the test script uses a comparison that might not be intuitive at first glance. It would be clearer to use a more explicit comparison for readability.

Workflow ID: wflow_eOExZTSDVU7JndPF

---

Want Ellipsis to fix these issues? Tag @ellipsis-dev in a comment. You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

[elijahbenizzy]

This will get easier if we drop 3.8 support, right? It's EOL officially now.