Skip to content

[MAINT] Run CIs on MNE-stable and update to latest Python version #303

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

tsbinns
Copy link
Collaborator

@tsbinns tsbinns commented May 23, 2025

Suggested by @larsoner. Currently only run unit tests on mne-main which can lead to bugs slipping through (e.g. #286).

This adds a test for the stable MNE version on Ubuntu. Should it be tested on more platforms like in MNE-BIDS?

Also updates the latest Python version tested from 3.12 to 3.13.

@tsbinns
Copy link
Collaborator Author

tsbinns commented May 23, 2025

Also, should this be updated to a Python version we're actually testing or just remove it?

- run: python -c "import mne; print(mne.datasets.testing.data_path(verbose=True))"
if: matrix.os == 'ubuntu-latest' && matrix.python-version == '3.11'

@larsoner
Copy link
Member

Also, should this be updated to a Python version we're actually testing or just remove it?

Ouch yes it means we've accidentally been skipping some tests I think. I would actually just get rid of the conditional, I think we always want the testing dataset

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