Skip to content

Conversation

@harryswift01
Copy link
Contributor

Summary

This PR separates MDAnalysis compatibility testing into its own workflow, updates how MDAnalysis versions are handled, expands OS coverage, and adds manual triggering support to the main CI workflow.

Changes

Move MDAnalysis compatibility tests into a dedicated workflow

  • Extracted the MDAnalysis compatibility matrix from project-ci.yaml.
  • Added a new workflow file: mdanalysis-compatibility.yaml.
  • Ensures MDAnalysis tests run independently from the main CI pipeline.

Adjust MDAnalysis version strategy

  • Based on feedback regarding instability with latest, pinned the version to 2.10.0.
  • Reduces risk of CI failures caused by upstream changes.
  • Renovate integration will be addressed in a separate PR.

Add OS matrix to MDAnalysis compatibility workflow

  • Added multi-platform testing to mdanalysis-compatibility.yaml:
    • ubuntu-24.04
    • windows-2025
    • macos-15
  • Ensures compatibility across all supported operating systems.

Add manual workflow dispatch to the main CI

  • Added workflow_dispatch to project-ci.yaml.
  • Allows the main CI to be triggered manually when needed.

Impact

  • Improves CI modularity by isolating MDAnalysis compatibility checks.
  • Reduces the risk of unexpected failures by pinning MDAnalysis to a stable version.
  • Enhances confidence in cross-platform support.
  • Provides developers with more flexibility via manual workflow triggering.
  • Makes the CI pipeline more predictable and maintainable overall.

@harryswift01 harryswift01 self-assigned this Nov 17, 2025
@harryswift01 harryswift01 added the bug Something isn't working label Nov 17, 2025
@harryswift01 harryswift01 linked an issue Nov 17, 2025 that may be closed by this pull request
@harryswift01 harryswift01 requested a review from jimboid November 17, 2025 11:26
Copy link
Member

@jimboid jimboid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is great Harry, separating out the MDA version checking from the main CI is going to help a lot with CI stability.

@harryswift01 harryswift01 merged commit e141b7d into main Nov 17, 2025
14 checks passed
@harryswift01 harryswift01 deleted the 181-use-latest-for-mdanalysis branch November 17, 2025 11:38
@harryswift01 harryswift01 restored the 181-use-latest-for-mdanalysis branch November 17, 2025 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Use 2.10.0 for MDAnalysis Compatibility Matrix in CI Pipeline

3 participants