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.

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

Add [pyoxidizer].lockfile #16537

Merged
merged 2 commits into from
Aug 16, 2022

Conversation

Eric-Arellano
Copy link
Contributor

I think that this was missing only due to oversight.

[ci skip-rust]
[ci skip-build-wheels]

# Rust tests and lints will be skipped. Delete if not intended.
[ci skip-rust]

# Building wheels and fs_util will be skipped. Delete if not intended.
[ci skip-build-wheels]
@sureshjoshi
Copy link
Member

100% oversight, I didn't even know about the lockfiles until I made the clang-format plugin.

@sureshjoshi
Copy link
Member

@Eric-Arellano I asked @chrisjrn about this in Slack, but might this also be a good time to bump the pyox version? It's about 3-4 minor versions old right now.

Only reason it's on 0.18 was because that what was out when I wrote this.

# Rust tests and lints will be skipped. Delete if not intended.
[ci skip-rust]

# Building wheels and fs_util will be skipped. Delete if not intended.
[ci skip-build-wheels]
@Eric-Arellano Eric-Arellano requested a review from benjyw August 16, 2022 17:16
@Eric-Arellano
Copy link
Contributor Author

I asked @chrisjrn about this in Slack, but might this also be a good time to bump the pyox version? It's about 3-4 minor versions old right now.

Sounds good to me. Probably better as a dedicated PR -- technically it's a User API Change.

Copy link
Member

@thejcannon thejcannon left a comment

Choose a reason for hiding this comment

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

Ideally #16541 ensures we don't hit this again.

@Eric-Arellano Eric-Arellano enabled auto-merge (squash) August 16, 2022 17:51
@Eric-Arellano Eric-Arellano merged commit c9c1659 into pantsbuild:main Aug 16, 2022
@Eric-Arellano Eric-Arellano deleted the pyoxidizer-lockfile branch August 16, 2022 17:59
cczona pushed a commit to cczona/pants that referenced this pull request Sep 1, 2022
I think that this was missing only due to oversight.

[ci skip-rust]
[ci skip-build-wheels]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants