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

Support trailing global flags in the native parser. #21480

Merged
merged 1 commit into from
Oct 3, 2024

Conversation

benjyw
Copy link
Sponsor Contributor

@benjyw benjyw commented Oct 2, 2024

The legacy parser supports having global flags at the end of
the command-line invocation, after all specs.

Now the native parser supports this too.

Fixes #21478

The legacy parser supports having global flags at the
end of the command-line invocation, after all specs.

Now the native parser supports this too.
@benjyw benjyw added needs-cherrypick category:bugfix Bug fixes for released features labels Oct 2, 2024
@benjyw benjyw added this to the 2.22.x milestone Oct 2, 2024
@benjyw benjyw requested a review from huonw October 2, 2024 22:01
@benjyw benjyw added the release-notes:not-required PR doesn't require mention in release notes label Oct 2, 2024
@benjyw benjyw merged commit d48b643 into pantsbuild:main Oct 3, 2024
25 checks passed
@benjyw benjyw deleted the fix_global_flag_native_parsing branch October 3, 2024 19:05
WorkerPants pushed a commit that referenced this pull request Oct 3, 2024
The legacy parser supports having global flags at the end of
the command-line invocation, after all specs.

Now the native parser supports this too.

Fixes #21478
WorkerPants pushed a commit that referenced this pull request Oct 3, 2024
The legacy parser supports having global flags at the end of
the command-line invocation, after all specs.

Now the native parser supports this too.

Fixes #21478
@WorkerPants
Copy link
Member

I tried to automatically cherry-pick this change back to each relevant milestone, so that it is available in those older releases of Pants.

✔️ 2.22.x

Successfully opened #21486.

✔️ 2.23.x

Successfully opened #21485.


Thanks again for your contributions!

🤖 Beep Boop here's my run link

benjyw added a commit that referenced this pull request Oct 3, 2024
…21480) (#21485)

The legacy parser supports having global flags at the end of
the command-line invocation, after all specs.

Now the native parser supports this too.

Fixes #21478

Co-authored-by: Benjy Weinberger <[email protected]>
benjyw added a commit that referenced this pull request Oct 3, 2024
…21480) (#21486)

The legacy parser supports having global flags at the end of
the command-line invocation, after all specs.

Now the native parser supports this too.

Fixes #21478

Co-authored-by: Benjy Weinberger <[email protected]>
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 release-notes:not-required PR doesn't require mention in release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Native option parser does pick up global options after path
3 participants