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.

Sign up for GitHub

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

Jump to bottom

"verbose &&" before @info "Scanning folder $folder for result files." #388

Closed
yp51md opened this issue May 8, 2023 · 2 comments
Closed

"verbose &&" before @info "Scanning folder $folder for result files." #388

yp51md opened this issue May 8, 2023 · 2 comments
Labels
good first issue Easy issue good for newcomers ux user experience

Comments

@yp51md
Copy link
Contributor

yp51md commented May 8, 2023

Sometimes I need to import several folders in a loop, every time I call collect_results(), Info like "@Info: Scanning folder" will be printed. It is kind of annoying. I am wondering if the "verbose &&" can be put in front of all the @info?

The @info:
https://github.com/JuliaDynamics/DrWatson.jl/blob/a9a238333a883935f6504382da28dc815fdec988/src/result_collection.jl#LL115C54-L115C54

Screenshot 2023-05-08 at 17 20 12
@Datseris Datseris added good first issue Easy issue good for newcomers ux user experience labels May 8, 2023
@Datseris
Copy link
Member

Datseris commented May 8, 2023

yes go ahead and do the pr!

@yp51md
Copy link
Contributor Author

yp51md commented May 8, 2023

done in this pr.

@yp51md yp51md closed this as completed May 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Easy issue good for newcomers ux user experience
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Datseris @yp51md