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

Fix formatting of the "pex.platforms is deprecated" message (Cherry-pick of #20514) #20527

Merged
merged 1 commit into from
Feb 13, 2024

Conversation

WorkerPants
Copy link
Member

Fixes a missing f-string formatter for the pex_binary's platforms field

Before:

16:02:15.59 [WARN] DEPRECATED: the platforms field is scheduled to be removed in version 2.22.0.dev0.
Using the platforms field in the target src/python/something_something:pex-executable. The platforms field is a hack. The abbreviated information it provides is sometimes insufficient, leading to hard-to-debug build issues. Use complete_platforms instead. See {doc_url('pex')} for details.

After:

16:03:43.09 [WARN] DEPRECATED: the platforms field is scheduled to be removed in version 2.22.0.dev0.
Using the platforms field in the target src/python/something_something:pex-executable. The platforms field is a hack. The abbreviated information it provides is sometimes insufficient, leading to hard-to-debug build issues. Use complete_platforms instead. See https://www.pantsbuild.org/v2.20/docs/pex for details.

Fixes a missing f-string formatter for the `pex_binary`'s `platforms`
field

Before:
> 16:02:15.59 [WARN] DEPRECATED: the platforms field is scheduled to be
removed in version 2.22.0.dev0.
Using the `platforms` field in the target
src/python/something_something:pex-executable. The platforms field is a
hack. The abbreviated information it provides is sometimes insufficient,
leading to hard-to-debug build issues. Use complete_platforms instead.
See {doc_url('pex')} for details.

After:
> 16:03:43.09 [WARN] DEPRECATED: the platforms field is scheduled to be
removed in version 2.22.0.dev0.
Using the `platforms` field in the target
src/python/something_something:pex-executable. The platforms field is a
hack. The abbreviated information it provides is sometimes insufficient,
leading to hard-to-debug build issues. Use complete_platforms instead.
See https://www.pantsbuild.org/v2.20/docs/pex for details.
@WorkerPants WorkerPants added this to the 2.19.x milestone Feb 12, 2024
@WorkerPants WorkerPants added the category:bugfix Bug fixes for released features label Feb 12, 2024
@WorkerPants WorkerPants requested a review from huonw February 12, 2024 05:17
@huonw huonw merged commit 3070214 into 2.19.x Feb 13, 2024
24 checks passed
@huonw huonw deleted the cherry-pick-20514-to-2.19.x branch February 13, 2024 01:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:bugfix Bug fixes for released features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants