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

Add testcore, testoptional deps to pyproject.toml #675

Merged
merged 9 commits into from
Jan 29, 2024

Conversation

akshayka
Copy link
Contributor

@akshayka akshayka commented Jan 27, 2024

  • To facilitate CI testing with minimal possible set of dependencies.
  • Refactor pandas/altair tests to only run when deps installed
  • Updates CI to run tests with and without the optional dependencies, to make sure we don't accidentally require an optional dep.

- To facilitate CI testing with minimal possible set of dependencies.
- Refactor pandas/altair tests to only run when deps installed
- TODO(in this change or future): update CI
Copy link

vercel bot commented Jan 27, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
marimo-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 29, 2024 8:25pm
marimo-storybook ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 29, 2024 8:25pm

Copy link

codecov bot commented Jan 27, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (30446ec) 69.36% compared to head (74e3091) 69.01%.
Report is 8 commits behind head on main.

❗ Current head 74e3091 differs from pull request most recent head 7607cfd. Consider uploading reports for the commit 7607cfd to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #675      +/-   ##
==========================================
- Coverage   69.36%   69.01%   -0.36%     
==========================================
  Files         139      139              
  Lines        7067     7067              
  Branches     1428     1428              
==========================================
- Hits         4902     4877      -25     
- Misses       1911     1938      +27     
+ Partials      254      252       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

mscolnick
mscolnick previously approved these changes Jan 27, 2024
@akshayka akshayka merged commit be01229 into main Jan 29, 2024
22 checks passed
@akshayka akshayka deleted the aka/test-optional-deps branch January 29, 2024 20:27
@github-actions github-actions bot locked and limited conversation to collaborators Jan 29, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants