Skip to content

Add colour to CI logs for readability#258

Merged
radarhere merged 1 commit intopython-pillow:mainfrom
hugovk:force_color
Feb 16, 2026
Merged

Add colour to CI logs for readability#258
radarhere merged 1 commit intopython-pillow:mainfrom
hugovk:force_color

Conversation

@hugovk
Copy link
Member

@hugovk hugovk commented Feb 16, 2026

No description provided.

COPY --from=0 /usr/local/include /usr/local/include
RUN yum install -y zlib-devel
COPY build.sh /build.sh
ENV FORCE_COLOR=1
Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member Author

Choose a reason for hiding this comment

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

The manylinux only build wheels, it doesn't run pytest directly.

The make test (line 36 of its Makefile) for this pulls $(TEST_IMAGE), which is pythonpillow/fedora-43-amd64.

After this PR is merged and pushed, pythonpillow/fedora-43-amd64 will have colour enabled, so its next run should have colour.

We could probably omit FORCE_COLOR=1 from the manylinux Dockerfiles, but I put it in for any other commands that can enable colour. Although I didn't check. Maybe the pip calls?

Copy link
Member

Choose a reason for hiding this comment

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

@radarhere radarhere merged commit c55f849 into python-pillow:main Feb 16, 2026
24 checks passed
@hugovk hugovk deleted the force_color branch February 17, 2026 05:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments