Skip to content

CI: have database builds *just* run db tests #39550

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

Closed
jreback opened this issue Feb 2, 2021 · 2 comments · Fixed by #39666
Closed

CI: have database builds *just* run db tests #39550

jreback opened this issue Feb 2, 2021 · 2 comments · Fixed by #39666
Labels
CI Continuous Integration
Milestone

Comments

@jreback
Copy link
Contributor

jreback commented Feb 2, 2021

The database builds: https://github.com/pandas-dev/pandas/pull/39457/checks?check_run_id=1797892949

take the longest. we should just have this run the database tests only. (so add a marker for the appropriate tests / just run the sql tests entirely).

@jreback jreback added the CI Continuous Integration label Feb 2, 2021
@jreback jreback added this to the 1.3 milestone Feb 2, 2021
@jreback
Copy link
Contributor Author

jreback commented Feb 2, 2021

cc @fangchenli

fangchenli added a commit to fangchenli/pandas that referenced this issue Feb 8, 2021
@fangchenli fangchenli mentioned this issue Feb 8, 2021
1 task
fangchenli added a commit to fangchenli/pandas that referenced this issue Feb 8, 2021
@fangchenli
Copy link
Member

cc @fangchenli

What shall we do for the cov build? If only run db test, we don't have the coverage anymore.

jreback pushed a commit that referenced this issue Mar 19, 2021
JulianWgs pushed a commit to JulianWgs/pandas that referenced this issue Jul 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous Integration
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants