Skip to content
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

feat: Scan files from a directory #55

Merged
merged 2 commits into from
Sep 14, 2022
Merged

feat: Scan files from a directory #55

merged 2 commits into from
Sep 14, 2022

Conversation

mgmacias95
Copy link
Member

closes #54 (comment)

@plusvic
Copy link
Member

plusvic commented Sep 12, 2022

The code looks good to me, but the documentation for the command is still vt scan file [file].. which suggests that the only thing that it accepts is a list of files. We should reflect the new use case in the documentation. The usage line may be something like vt scan file [[dir] | [file]..], and we need to change the description as well, which currently says:

Scan one or more files.

This command receives one or more file paths and uploads them to VirusTotal for
scanning....

@mgmacias95
Copy link
Member Author

PTAL

@mgmacias95 mgmacias95 merged commit 331af40 into master Sep 14, 2022
@mgmacias95 mgmacias95 deleted the scan_dirs branch September 14, 2022 14:05
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.

How to scan contents of a folder?
2 participants