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

test: allow testing single dependency (without other installed) #1843

Closed
wants to merge 10 commits into from

Conversation

FBruzzesi
Copy link
Member

What type of PR is this? (check all applicable)

  • πŸ’Ύ Refactor
  • ✨ Feature
  • πŸ› Bug Fix
  • πŸ”§ Optimization
  • πŸ“ Documentation
  • βœ… Test
  • 🐳 Other

Related issues

Checklist

  • Code follows style guide (ruff)
  • Tests added
  • Documented the changes

If you have comments or can explain your changes, please do so below

@FBruzzesi
Copy link
Member Author

It was worth trying πŸ˜‚πŸ˜‡

@FBruzzesi FBruzzesi closed this Jan 20, 2025
@MarcoGorelli
Copy link
Member

It was worth trying πŸ˜‚πŸ˜‡

πŸ˜„ what went wrong?

@FBruzzesi
Copy link
Member Author

πŸ˜„ what went wrong?

I might try again in the future, but different tests are failing for different reasons

@MarcoGorelli
Copy link
Member

sure! looks like the no_imports_test test was failing, just adding , raising=False) to all delitem calls should be enough to address that

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.

test: allow for tests to be run without Polars / PyArrow/ pandas installed
2 participants