Skip to content

Convert get-subsystem-split to git-subsystem-split.py #30783

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

peterbarker
Copy link
Contributor

Thanks ChatGPT!

Perhaps we have this as an alternative for testing for some time, then rename it over the existing shell script.

@tpwrules
Copy link
Contributor

argparse supports all of the manually-implemented argument logic, e.g. https://docs.python.org/3/library/argparse.html#mutual-exclusion

@peterbarker
Copy link
Contributor Author

argparse supports all of the manually-implemented argument logic, e.g. https://docs.python.org/3/library/argparse.html#mutual-exclusion

I assure you no logic in this PR was made manually.

I want agreement that we'll do this before working on it.

@tpwrules
Copy link
Contributor

Sure, I'd approve more Python over Bash. This would let us address various limitations more easily and testably. But I would not deign to approve stuff written by ChatGPT ;)

@peterbarker
Copy link
Contributor Author

But I would not deign to approve stuff written by ChatGPT ;)

Which is why I clearly marked this as such and said it needed more testing :-)

OTOH, it appears to have done a damned good job on first inspection (OTGH that's often the case with this stuff!).

@tridge
Copy link
Contributor

tridge commented Aug 2, 2025

I'd definitely like to see it in python

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.

3 participants