Skip to content

TST Create dedicated dataframe / feature count tests category #29713

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

Draft
wants to merge 19 commits into
base: main
Choose a base branch
from

Conversation

adrinjalali
Copy link
Member

Requires #29702 and #29699 to substantially reduce the diff.

This creates a dedicated dataframe test category, moving the tests from test_common to estimator_checks to include them for all third party estimators as well, which can now be disabled easily by disabling the flag when calling tests.

@adrinjalali adrinjalali added the Developer API Third party developer API related label Aug 25, 2024
Copy link

github-actions bot commented Aug 25, 2024

✔️ Linting Passed

All linting checks passed. Your pull request is in excellent shape! ☀️

Generated for commit: b4c3095. Link to the linter CI: here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Developer API Third party developer API related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant