Skip to content
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

BLD: Add CI runner for core test suite (no optional dependencies) #1314

Merged
merged 2 commits into from
Oct 13, 2020

Conversation

mdickinson
Copy link
Member

This PR adds a GitHub Actions workflow for running the test suite with none of the optional dependencies present in the environment.

The idea is to catch bugs where either a test or (worse) a piece of the core Traits functionality accidentally depends on another package being present.

It seems unnecessary to runs these tests on all possible combinations of Python version and platform (which would give 12 jobs), but given that there's potential for dependency bugs to be both platform-specific and Python-version specific, we compromise by testing only on Windows and Linux, and only on Python 3.6 and Python 3.9, for a total of four jobs.

Closes #1299.

Copy link
Contributor

@rahulporuri rahulporuri left a comment

Choose a reason for hiding this comment

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

LGTM

@mdickinson mdickinson changed the title Add CI runner for core test suite (no optional dependencies) BLD: Add CI runner for core test suite (no optional dependencies) Oct 13, 2020
@mdickinson mdickinson merged commit a9dc7cf into master Oct 13, 2020
@mdickinson mdickinson deleted the build/bare-traits-build branch October 13, 2020 12:37
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.

Add test run for Traits with no extra packages installed
2 participants