Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

🧪🐛 Make coverage use upstream Cython plugin #538

Merged
merged 1 commit into from
Nov 28, 2023
Merged

🧪🐛 Make coverage use upstream Cython plugin #538

merged 1 commit into from
Nov 28, 2023

Conversation

webknjaz
Copy link
Member

@webknjaz webknjaz added the dependencies Pull requests that update a dependency file label Nov 28, 2023
@webknjaz webknjaz self-assigned this Nov 28, 2023
@packit-as-a-service 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-538
  • And now you can install the packages.

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

@webknjaz webknjaz force-pushed the maintenance/gha-upstream-cython-coverage branch from ac90b87 to a36d965 Compare November 28, 2023 21:24
@webknjaz webknjaz marked this pull request as ready for review November 28, 2023 21:33
@webknjaz webknjaz force-pushed the maintenance/gha-upstream-cython-coverage branch from a36d965 to 1affefe Compare November 28, 2023 22:59
@webknjaz webknjaz force-pushed the maintenance/gha-upstream-cython-coverage branch from 1affefe to fe06f9a Compare November 28, 2023 23:00
@psf-chronographer psf-chronographer bot added the bot:chronographer:provided There is a change note present in this PR label Nov 28, 2023
@webknjaz
Copy link
Member Author

While the CI is red, I confirmed through the jobs that succeed that this patch does what it's supposed to.

@webknjaz webknjaz merged commit 8de65c7 into ansible:devel Nov 28, 2023
148 of 156 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@webknjaz webknjaz

Labels
bot:chronographer:provided There is a change note present in this PR dependencies Pull requests that update a dependency file
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[TODO] Switch to using built-in Cython coverage plugins once it's fixed upstream
1 participant
@webknjaz