search

idank / explainshell

match command-line arguments to their help text
GNU General Public License v3.0
13.29k stars 794 forks source link

Migrate from Python 2.7 to Python 3.12 #330

Open mundanevision20 opened 2 months ago

mundanevision20 commented 2 months ago

While I mainly adapted the Python code to work with Python 3.12 I also enhanced the formatting and logging to ease future maintenance.

mundanevision20 commented 2 months ago

I also merged #237 by @rugk, #248 by @apoorvlathey and #293 by @Strahinja into this PR as the original PRs are apparently no longer active.

Based on #232 by @wesinator I updated the references to use manpages of Ubuntu noble (24.04 LTS). Due to the high number of merge conflicts I decided to apply the changes without merging.

idank commented 2 months ago

Thanks for this work! I suggest to keep this PR focused on the python migration and not pull in unrelated stuff like updating the man pages.

On Sat, Sep 21, 2024, 13:51 Henning Weller @.***> wrote:

I also merged #237 https://github.com/idank/explainshell/pull/237 by @rugk https://github.com/rugk, #248 https://github.com/idank/explainshell/pull/248 by @apoorvlathey https://github.com/apoorvlathey and #293 https://github.com/idank/explainshell/pull/293 by @Strahinja https://github.com/Strahinja into this PR as the original PRs are apparently no longer active.

Based on #232 https://github.com/idank/explainshell/pull/232 by @wesinator https://github.com/wesinator I updated the references to use manpages of Ubuntu noble (24.04 LTS). Due to the high number of merge conflicts I decided to apply the changes without merging.

— Reply to this email directly, view it on GitHub https://github.com/idank/explainshell/pull/330#issuecomment-2365142137, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAJX5RCJB7K2V7NGZTHM7IDZXVFU3AVCNFSM6AAAAABOTOTIV6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNRVGE2DEMJTG4 . You are receiving this because you are subscribed to this thread.Message ID: @.***>