Skip to content

Conversation

aignas
Copy link
Collaborator

@aignas aignas commented Oct 12, 2023

This class is for being able to more easily recreate a wheel file after
extracting it. This is not intended for usage outside the rules_python
project. Also stop sorting the entries when writing a RECORD file making
the order of the RECORD file to be the same as the order the files to
the zip file are added.

Towards #1076

Copy link
Collaborator

@groodt groodt left a comment

Choose a reason for hiding this comment

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

LGTM

I'm not entirely sure what our preference is in terms of: #1453

@aignas
Copy link
Collaborator Author

aignas commented Oct 16, 2023

I would like to merge #1453 and then rebase and merge this.

@groodt
Copy link
Collaborator

groodt commented Oct 16, 2023

I would like to merge #1453 and then rebase and merge this.

SGTM

This class is for being able to more easily recreate a wheel file after
extracting it. This is not intended for usage outside the rules_python
project.

Towards bazel-contrib#1076
@aignas aignas force-pushed the feat/1076/whlmaker-whlfile branch from 71d508e to 0d2f7f0 Compare October 17, 2023 05:23
@aignas aignas marked this pull request as ready for review October 17, 2023 05:24
@aignas aignas requested a review from rickeylev as a code owner October 17, 2023 05:24
@aignas aignas changed the title refactor(whlmaker): introduce an internal _WhlFile class feat(whlmaker): introduce an internal _WhlFile class and stop sorting RECORD Oct 17, 2023
@aignas aignas enabled auto-merge October 17, 2023 05:33
@aignas aignas added this pull request to the merge queue Oct 17, 2023
Merged via the queue into bazel-contrib:main with commit e3a93f3 Oct 17, 2023
@aignas aignas deleted the feat/1076/whlmaker-whlfile branch May 13, 2024 06:48
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