Skip to content

docs: sync FLAGS.md with actual CLI arguments - #455

Closed
SpiliosDimakopoulos wants to merge 1 commit into
kaifcodec:mainfrom
SpiliosDimakopoulos:docs/fix-flags-table
Closed

docs: sync FLAGS.md with actual CLI arguments#455
SpiliosDimakopoulos wants to merge 1 commit into
kaifcodec:mainfrom
SpiliosDimakopoulos:docs/fix-flags-table

Conversation

@SpiliosDimakopoulos

Copy link
Copy Markdown
Contributor

docs/FLAGS.md was out of sync with the actual argparse definitions in
main.py:

  • Removed -p, --permute: this flag doesn't exist anywhere in the
    codebase (verified via repo-wide grep). Pattern-based permutations are
    generated inline via -u/-e (e.g. -u "john[a-c]"), as already shown
    correctly elsewhere in this same file and in the README.
  • Added -U, --update and --version: both exist in main.py's
    argparse but were missing from this table.

Docs only, no code changes.

Removed -p/--permute, which no longer exists anywhere in __main__.py (patterns are now used inline with -u/-e, e.g. -u 'john[a-c]'). Added -U/--update and --version, which exist in the CLI but were missing from this table.
@VamatoHD

Copy link
Copy Markdown
Collaborator

Removed -p, --permute: this flag doesn't exist anywhere in the
codebase (verified via repo-wide grep). Pattern-based permutations are
generated inline via -u/-e (e.g. -u "john[a-c]"), as already shown
correctly elsewhere in this same file and in the README.

Even though you stated this, this change wasn't implemented in this PR.

@kaifcodec

Copy link
Copy Markdown
Owner

@VamatoHD Seems like the PR author is not going to respond on these also there are open questions and change requested in #452 as well 2 days ago.
I will convert these PRs to draft if I don't recieve any response.

@kaifcodec

Copy link
Copy Markdown
Owner

This PR will be covered in #459 .
Closing it.

@kaifcodec kaifcodec closed this Jul 28, 2026
@kaifcodec kaifcodec added the duplicate This issue or pull request already exists label Jul 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

duplicate This issue or pull request already exists

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants