Skip to content

Drop the separate core test workflow - #4822

Open
h-mayorquin wants to merge 1 commit into
SpikeInterface:mainfrom
h-mayorquin:remove_core_tests
Open

h-mayorquin wants to merge 1 commit into
SpikeInterface:mainfrom
h-mayorquin:remove_core_tests

Conversation

@h-mayorquin

Copy link
Copy Markdown
Contributor

core-test.yml runs the core tests on the three OSes with Python 3.10, but all-tests.yml already does the same as its first test step (same events, same --group test-core install, plus Python 3.13) and stops the job before the expensive modules if core fails. This PR removes core-test.yml so every push to a PR runs three fewer jobs, and runs the core step of all-tests.yml through run_tests.sh so we keep the verbose output and the timing summary.
This is the last item of #3100 that I think is worth doing, so this should close #3100.

@h-mayorquin h-mayorquin self-assigned this Oct 2, 2026
@h-mayorquin h-mayorquin added testing Related to test routines continuous integration Related to CI labels Oct 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

continuous integration Related to CI testing Related to test routines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

One CI script to rule them all. Remaining improvements to testing

1 participant