Skip to content

Update test; use lazy import#1267

Merged
lrandersson merged 1 commit into
conda:mainfrom
lrandersson:dev-ra-fix-canary-ubuntu-test
Jun 22, 2026
Merged

Update test; use lazy import#1267
lrandersson merged 1 commit into
conda:mainfrom
lrandersson:dev-ra-fix-canary-ubuntu-test

Conversation

@lrandersson

@lrandersson lrandersson commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

Description

I noticed this test failed: https://github.com/conda/constructor/actions/runs/27795032460/job/82713863768

ModuleNotFoundError: No module named 'PIL'

I recall we discussed this in a separate PR, because why don't the other linux tests fail when we have - pillow >=3.1 # [osx or win]? To my understanding that selector is a conda-build recipe selector and is not honored in the other jobs - for example looking at the other linux jobs you can see in the logs that pillow is installed.

That being said, I updated the test to do a lazy import since the test is a Windows test anyway.

Checklist - did you ...

  • Add a file to the news directory (using the template) for the next release's release notes?
  • Add / update necessary tests?
  • Add / update outdated documentation?

@lrandersson lrandersson self-assigned this Jun 22, 2026
@lrandersson lrandersson requested a review from a team as a code owner June 22, 2026 12:57
@github-project-automation github-project-automation Bot moved this to 🆕 New in 🔎 Review Jun 22, 2026
@conda-bot conda-bot added the cla-signed [bot] added once the contributor has signed the CLA label Jun 22, 2026
@github-project-automation github-project-automation Bot moved this from 🆕 New to ✅ Approved in 🔎 Review Jun 22, 2026
@lrandersson lrandersson merged commit aa23672 into conda:main Jun 22, 2026
33 of 35 checks passed
@github-project-automation github-project-automation Bot moved this from ✅ Approved to 🏁 Done in 🔎 Review Jun 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed [bot] added once the contributor has signed the CLA

Projects

Status: 🏁 Done

Development

Successfully merging this pull request may close these issues.

3 participants