[CI] remove unused import in python tests #8409
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
👋 I'm interested in spending some time helping out on
xgboost
, so I started looking around tonight.I ran
flake8
over the Python package tonight, and found what looks like an unused import.This PR proposes removing it.
How I tested this
locally on my mac (click for details)
Basically copied what I saw in
xgboost/.github/workflows/python_tests.yml
Line 157 in 0db903b
Created and activated a
conda
environment.mamba env create \ --name xgboost-dev \ pythoon=3.10 \ pandas \ numpy \ scipy \ dask \ distributed \ hypothesis \ pytest \ pytest-cov \ matplotlib source activate xgboost-dev
Then compiled, built the Python package, installed it, and ran the Python tests.
This resulted in 3 test failures which I'm guessing are not related to the change in this PR.
Notes for Reviewers
Thanks very much for your time and consideration.