Skip to content

cmdline-opts: category cleanup #14101

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

Merged
merged 1 commit into from
Jul 5, 2024
Merged

cmdline-opts: category cleanup #14101

merged 1 commit into from
Jul 5, 2024

Conversation

bagder
Copy link
Member

@bagder bagder commented Jul 4, 2024

the curl category

--create-dirs removed from 'curl'
--ftp-create-dirs removed from 'curl'
--netrc moved to 'auth' from 'curl'
--netrc-file moved to 'auth' from 'curl'
--netrc-optional moved to 'auth' from 'curl'
--no-buffer moved to 'output' from 'curl'
--no-clobber removed from 'curl'
--output removed from 'curl'
--output-dir removed from 'curl'
--remove-on-error removed from 'curl'

Created global category

Made all "global" options set this category

Created deprecated category

Moved deprecated options to it.

Removed ECH category

It should not be a category. Had only a single option.

Removed misc category

It should not be a category.

--use-ascii moved to ftp and output
--xattr moved to output
--service-name moved to auth

Removed ipfs category

It should not be a category. Had only a single option.

managen

  • errors if an option is given a category name that is not already setup for in code
  • verifies that options set scope: global also is put in category `global´

@bagder bagder changed the title cmdline-opts: cleanup the 'curl' category cmdline-opts: category cleanup Jul 4, 2024
@github-actions github-actions bot added the tests label Jul 4, 2024
@bagder bagder force-pushed the bagder/categories branch 2 times, most recently from 8722e82 to 79d810e Compare July 4, 2024 22:02
@bagder bagder force-pushed the bagder/categories branch from 43d1a60 to fd0fd7d Compare July 5, 2024 06:48
Option cleanups:

 --get is not upload
 --form* are post
 - added several options into ldap, smtp, imap and pop3
 - shortened the category descriptions in the list

category curl fixes:

 --create-dirs removed from 'curl'
 --ftp-create-dirs removed from 'curl'
 --netrc moved to 'auth' from 'curl'
 --netrc-file moved to 'auth' from 'curl'
 --netrc-optional moved to 'auth' from 'curl'
 --no-buffer moved to 'output' from 'curl'
 --no-clobber removed from 'curl'
 --output removed from 'curl'
 --output-dir removed from 'curl'
 --remove-on-error removed from 'curl'

Add a "global" category:

- Made all "global" options set this category

Add a "deprecated" category:

- Moved the deprecated options to it (maybe they should not be in any
 category long term)

Add a 'timeout' category

- Put a number of appropriate options in it

Add an 'ldap' category

- Put the LDAP related option in there

Remove categories "ECH" and "ipfs"

- They should not be categories. Had only one single option each.

Remove category "misc"

- It should not be a category as it is impossible to know when to browse
  it.

--use-ascii moved to ftp and output
--xattr moved to output
--service-name moved to auth

Managen fixes:

- errors if an option is given a category name that is not already setup
  for in code

- verifies that options set `scope: global` also is put in category
  `global´

Closes #14101
@bagder bagder force-pushed the bagder/categories branch from fd0fd7d to 2abfc75 Compare July 5, 2024 09:06
@bagder bagder closed this in 2abfc75 Jul 5, 2024
@bagder bagder merged commit 2abfc75 into master Jul 5, 2024
166 of 173 checks passed
@bagder bagder deleted the bagder/categories branch July 5, 2024 11:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

1 participant