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

Merge 23.7.x branch back into main #12921

Merged
merged 3 commits into from
Jul 25, 2023
Merged

Merge 23.7.x branch back into main #12921

merged 3 commits into from
Jul 25, 2023

Conversation

kenodegard
Copy link
Contributor

@kenodegard kenodegard commented Jul 25, 2023

Description

Merges release branch (with changelog changes, plugins disabling patch, and 2-phase parser/greedy subparser) back into main.

Xref #12849

Warning
DO NOT SQUASH MERGE!

Checklist - did you ...

  • Add a file to the news directory (using the template) for the next release's release notes?
  • Add / update necessary tests?
  • Add / update outdated documentation?

jezdez and others added 3 commits July 19, 2023 23:22
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Ken Odegard <kodegard@anaconda.com>
Co-authored-by: Bianca Henderson <bhenderson@anaconda.com>
Implement a 2-phase parser so we can properly intercept `--no-plugins` before loading the subcommand plugins. In addition to the 2-phase parser we add a new subparser that greedily consumes all unknown arguments to be passed to the subcommand plugins. These changes allows a simplification of how third-party subcommands are detected and displayed in the help text (whether subcommand plugins or legacy executables) so all subcommands are now processed via conda's argparse parser instead of hijacking argparse's error handling.

---------

Co-authored-by: Jannis Leidel <jannis@leidel.info>
Co-authored-by: Ken Odegard <kodegard@anaconda.com>
Co-authored-by: Jannis Leidel <jannis@leidel.info>
@kenodegard kenodegard requested a review from a team as a code owner July 25, 2023 15:47
@kenodegard kenodegard mentioned this pull request Jul 25, 2023
62 tasks
@conda-bot conda-bot added the cla-signed [bot] added once the contributor has signed the CLA label Jul 25, 2023
@kenodegard kenodegard merged commit 0ac1716 into main Jul 25, 2023
140 checks passed
@jezdez jezdez mentioned this pull request Sep 26, 2023
92 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed [bot] added once the contributor has signed the CLA
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

4 participants