Skip to content

Conversation

@r41k0u
Copy link
Collaborator

@r41k0u r41k0u commented Nov 11, 2025

No description provided.

@r41k0u r41k0u marked this pull request as ready for review November 11, 2025 11:52
@r41k0u r41k0u requested a review from Copilot November 11, 2025 11:53
Copilot finished reviewing on behalf of r41k0u November 11, 2025 11:54
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 enhances the CI/CD workflows and build configuration for the pylibbpf project. Despite the title "Print pylibbpf dir in test", the changes primarily focus on improving build reliability and test coverage across different Python versions.

Key changes:

  • Updated CI workflows to use recursive submodule checkout and enhanced verification steps
  • Modified Python version support from 3.11 to 3.12-3.13 in wheel building
  • Added setuptools package configuration in pyproject.toml

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

File Description
setup.py Removed zip_safe=False parameter from setup call
pyproject.toml Added setuptools package discovery and package-data configuration
.github/workflows/wheels.yml Updated submodule checkout, Python versions (3.12-3.13), enhanced test command with BpfObject assertion, and added build dependencies
.github/workflows/pip.yml Restructured workflow with detailed diagnostic steps, recursive submodule checkout, updated dependencies, and comprehensive build verification

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@r41k0u r41k0u changed the title Print pylibbpf dir in test Fix GH Actions, make Python3.12 the oldest supported version Nov 11, 2025
@r41k0u r41k0u merged commit 270f433 into master Nov 11, 2025
6 checks passed
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