Skip to content

Don't add dependency for py-pytest#189

Merged
Taka-Kazu merged 1 commit intomasterfrom
do-not-add-dependency-for-py-pytest
Jan 9, 2026
Merged

Don't add dependency for py-pytest#189
Taka-Kazu merged 1 commit intomasterfrom
do-not-add-dependency-for-py-pytest

Conversation

@Taka-Kazu
Copy link
Copy Markdown
Contributor

@Taka-Kazu Taka-Kazu commented Jan 7, 2026

This part is added by #158, but I don't remember why this is necessary.
For Alpine 3.23, this causes conflicts of version between py3-pytest7 and py3-pytest(8).

pytest7 is required for current ROS 2 python packages, so there is no need to install py3-pytest(8) at default.(This is same for 3.20)

Copy link
Copy Markdown
Member

@at-wat at-wat left a comment

Choose a reason for hiding this comment

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

LGTM

@Taka-Kazu Taka-Kazu merged commit cfc2786 into master Jan 9, 2026
4 checks passed
@Taka-Kazu Taka-Kazu deleted the do-not-add-dependency-for-py-pytest branch January 9, 2026 05:22
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