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

Temporarily constraint to xgboost<1.0.0. #948

Merged
merged 1 commit into from
Feb 21, 2020

Conversation

hvy
Copy link
Member

@hvy hvy commented Feb 20, 2020

Temporarily constraint to xgboost<1.0.0 since 1.0.0 seems to contain accidental f-strings which cause our Python 3.5 CI builds to fail.

Should be reverted after dmlc/xgboost#5328 is addressed.

@hvy hvy added the CI Continuous integration. label Feb 20, 2020
@hvy hvy added this to the v1.2.0 milestone Feb 20, 2020
@hvy hvy force-pushed the hotfix-xgboost-py35-f-string branch from 73f3922 to bb48c47 Compare February 20, 2020 12:40
@hvy
Copy link
Member Author

hvy commented Feb 20, 2020

Another PR that's related to xgboost==1.0.0 #947. This PR should probably be merged before.

@codecov-io
Copy link

Codecov Report

Merging #948 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #948   +/-   ##
=======================================
  Coverage   90.23%   90.23%           
=======================================
  Files         112      112           
  Lines        9292     9292           
=======================================
  Hits         8385     8385           
  Misses        907      907
Impacted Files Coverage Δ
setup.py 0% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update eba96e7...bb48c47. Read the comment docs.

Copy link
Member

@toshihikoyanase toshihikoyanase left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you for your swift action!

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 this pull request may close these issues.

3 participants