Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 12, 2025

The CLI interface had inconsistent flag naming—some used hyphens (--skip-if-downloaded-already) while others used underscores (--output_folder, --aspera_maximum_bandwidth).

Changes

  • CLI flags: Converted all multi-word options to use hyphens consistently:

    • --output_folder--output-folder
    • --file_name--file-name
    • --aspera_maximum_bandwidth--aspera-maximum-bandwidth
    • --checksum_check--checksum-check
    • --output_file--output-file
    • --page_size--page-size
    • --sort_direction--sort-direction
    • --sort_fields--sort-fields
  • Documentation: Updated README examples to reflect hyphenated flags

Example

# Before
pridepy download-all-public-raw-files -a PXD012353 -o /path/ --aspera_maximum_bandwidth 100M

# After
pridepy download-all-public-raw-files -a PXD012353 -o /path/ --aspera-maximum-bandwidth 100M

Note: Python function parameters remain unchanged (underscored) as Click automatically converts hyphens to underscores for internal use.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI mentioned this pull request Dec 12, 2025
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 12, 2025

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Comment @coderabbitai help to get the list of available commands and usage tips.

@ypriverol ypriverol marked this pull request as ready for review December 12, 2025 11:35
@qodo-code-review
Copy link
Contributor

PR Code Suggestions ✨

No code suggestions found for the PR.

Co-authored-by: ypriverol <52113+ypriverol@users.noreply.github.com>
Copilot AI changed the title [WIP] Update skip options for consistency with hyphens Standardize CLI flags to use hyphens instead of underscores Dec 12, 2025
Copilot AI requested a review from ypriverol December 12, 2025 11:42
@ypriverol ypriverol added the enhancement New feature or request label Dec 12, 2025
@ypriverol ypriverol merged commit aecc9ac into dev Dec 12, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants