Skip to content

Conversation

LunaWhispers
Copy link
Contributor

Description

Add --recursive argrument to download directory files.
Specify whether to download the directory recursively.
If it is true, dfget will download all files in the directory.
If it is false, dfget will download the single file specified by the URL.

if you need download single file, don't use --recursive argrument.

for example:

dfget http://<url><path>:<port> -O <output-dir> --recursive

Related Issue

Motivation and Context

Screenshots (if appropriate)

Signed-off-by: LunaWhispers <yangmuyucs@gmail.com>
Copy link

codecov bot commented Aug 18, 2025

Codecov Report

❌ Patch coverage is 95.45455% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 48.73%. Comparing base (d4aaa03) to head (acd2d3c).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
dragonfly-client/src/bin/dfget/main.rs 95.45% 3 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1290      +/-   ##
==========================================
+ Coverage   48.58%   48.73%   +0.15%     
==========================================
  Files          60       60              
  Lines       15002    15068      +66     
==========================================
+ Hits         7288     7344      +56     
- Misses       7714     7724      +10     
Files with missing lines Coverage Δ
dragonfly-client/src/bin/dfget/main.rs 50.51% <95.45%> (+2.47%) ⬆️

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@LunaWhispers LunaWhispers added the enhancement New feature or request label Aug 18, 2025
@gaius-qi
Copy link
Member

Signed-off-by: LunaWhispers <yangmuyucs@gmail.com>
Signed-off-by: LunaWhispers <yangmuyucs@gmail.com>
Signed-off-by: LunaWhispers <yangmuyucs@gmail.com>
Signed-off-by: LunaWhispers <yangmuyucs@gmail.com>
Signed-off-by: LunaWhispers <yangmuyucs@gmail.com>
gaius-qi
gaius-qi previously approved these changes Aug 18, 2025
Copy link
Member

@gaius-qi gaius-qi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Signed-off-by: LunaWhispers <yangmuyucs@gmail.com>
Copy link
Member

@gaius-qi gaius-qi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@chlins chlins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@chlins chlins merged commit 205e0e9 into dragonflyoss:main Aug 18, 2025
7 checks passed
@LunaWhispers LunaWhispers deleted the feature/recursive branch August 28, 2025 07:33
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.

3 participants