Skip to content

Conversation

@webknjaz
Copy link
Member

It was originally removed from other places in #658. But the cibuildwheel was overlooked. This patch corrects that.

It was originally removed from other places in ansible#658. But the
`cibuildwheel` was overlooked. This patch corrects that.
@webknjaz webknjaz self-assigned this Aug 21, 2025
@psf-chronographer psf-chronographer bot added the bot:chronographer:provided There is a change note present in this PR label Aug 21, 2025
@webknjaz webknjaz marked this pull request as ready for review August 21, 2025 11:29
Copilot AI review requested due to automatic review settings August 21, 2025 11:29
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes the pytest-forked dependency from the cibuildwheel configuration to complete the cleanup that was previously done in #658 but missed this location.

  • Removes pytest-forked from the test-requires configuration in cibuildwheel
  • Adds a changelog fragment documenting the packaging change

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
pyproject.toml Removes pytest-forked from cibuildwheel test requirements
docs/changelog-fragments/760.packaging.rst Adds changelog entry for the packaging change

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@packit-as-a-service
Copy link

Congratulations! One of the builds has completed. 🍾

You can install the built RPMs by following these steps:

  • sudo yum install -y dnf-plugins-core on RHEL 8
  • sudo dnf install -y dnf-plugins-core on Fedora
  • dnf copr enable packit/ansible-pylibssh-760
  • And now you can install the packages.

Please note that the RPMs should be used only in a testing environment.

@webknjaz webknjaz merged commit 9de10a3 into ansible:devel Aug 21, 2025
41 of 45 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bot:chronographer:provided There is a change note present in this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant