Skip to content

Commit

Permalink
becca's comments
Browse files Browse the repository at this point in the history
  • Loading branch information
MichaelFu512 committed May 22, 2024
1 parent cd3f4f8 commit 2b11a69
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/install_test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ jobs:
run: brew install cmake libomp
- name: Build evalml package
run: make package
- name: Install evalml from sdist (not using cache)
- name: Install evalml from sdist
run: |
python -m pip install "unpacked_sdist/."
- name: Test by importing packages
Expand Down
2 changes: 1 addition & 1 deletion evalml/tests/utils_tests/test_gen_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@
def in_container_arm64():
"""Helper fixture to run chromium as a single process for kaleido.
Useful as kaleido > 0.1.0 on windows seems to plotly.write_image to hang indefinitely.
Useful as kaleido > 0.1.0 can cause plotly.write_image to hang indefinitely.
Makes tests that use plotly not hang and thus pass.
"""
import plotly.io as pio

Check warning on line 40 in evalml/tests/utils_tests/test_gen_utils.py

View check run for this annotation

Codecov / codecov/patch

evalml/tests/utils_tests/test_gen_utils.py#L40

Added line #L40 was not covered by tests
Expand Down

0 comments on commit 2b11a69

Please sign in to comment.