Extract CLI logic from `clinic.py`

Argument-Clinic / cpython

The Python programming language

https://www.python.org/

Other

1 stars 0 forks source link

Closed AlexWaygood closed 8 months ago

AlexWaygood commented 10 months ago

Move Tools/clinic/clinic.py and Tools/clinic/cpp.py inside the Tools/clinic/libclinic package.
Extract the CLI logic from clinic.py into a new libclinic submodule, libclinic.cli
Add a new file, Tools/clinic/run_clinic.py, which simply imports libclinic.cli and executes the CLI if __name__ == "__main__". Update the Makefile etc. so that they run Tools/clinic/run_clinic.py instead of Tools/clinic/clinic.py.

erlend-aasland commented 10 months ago

Nice

AlexWaygood commented 10 months ago

(I'll probably not get back to this until 27th now — will upstream this then :)

erlend-aasland commented 10 months ago

(I'll probably not get back to this until 27th now — will upstream this then :)

Enjoy the days off 🎅🎄🎉

erlend-aasland commented 10 months ago

Hm, I think parse_file() should live where the CLI lives; it is an app function, not a library function.

erlend-aasland commented 8 months ago

What do you think of this, @vstinner?

erlend-aasland commented 8 months ago

Upstream as python/cpython#115542