Skip to content

Allow CLI output to a new path - #525

Merged
rocky merged 1 commit into
rocky:masterfrom
dajiaohuang:fix/output-path-option
Sep 29, 2026
Merged

rocky merged 1 commit into
rocky:masterfrom
dajiaohuang:fix/output-path-option

Conversation

@dajiaohuang

Copy link
Copy Markdown
Contributor

The output option is documented for writing decompiled results to a file or directory, and the implementation creates missing destinations. The CLI currently rejects any new destination during argument parsing because the option requires the path to exist.

Allowing missing output paths restores the documented behavior while retaining file and directory validation for existing paths. Added a regression test covering a new output file path.

Validation:

  • python -m compileall -q uncompyle6 pytest/test_cli.py
  • click path conversion accepts a new output path
  • git diff --check

@rocky

rocky commented Sep 25, 2026 •

Copy link
Copy Markdown
Owner

@dajiaohuang I had hoped to be able to look at this and the other PR this week.

Unfortunately, my schedule right now is that I can't look at it until sometime next week.

@rocky
rocky merged commit e064cbe into rocky:master Sep 29, 2026
3 checks passed
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.

2 participants