pacman icon indicating copy to clipboard operation
pacman copied to clipboard

Published 20 hours ago • verified publisher icon trinker

Add verbose argument

Open trinker opened this issue 11 years ago • 6 comments

p_install and p_load are verbose. Add a verbose argument that suppesses these messages.

verbose = getOption("pac_vervose")

Defaulting to TRUE and allowing the user to set this to FALSE in their .Rprofile

trinker avatar Feb 09 '14 04:02 trinker

The issue #40 you reference seems to have disappeared from p_load. Why?

I started using quiet = FALSE as an argument before I saw this (in p_delete; https://github.com/trinker/pacman/issues/11#issuecomment-46742207) as I grepped and found no verbose argument in pacman as library's quiet argument is closer to our intent. Yihui uses this in knitr as well. Thoughts?

trinker avatar Jun 21 '14 03:06 trinker

p_load is now quiet. But sometimes I'd like to see package load messages and conflicts. Is this no longer an option? Thanks guys!

wildmolasses avatar Jul 13 '16 18:07 wildmolasses

It is not currently.

trinker avatar Jul 13 '16 19:07 trinker

Agree this will be great. Could we have an option quiet = FALSER / TRUE in p_load and p_unload ?

Fablepongiste avatar Oct 12 '16 17:10 Fablepongiste

It is really a small change in code but would be quite useful ? @trinker Should I do it and push it ? Or is it something you plan to do ?

Fablepongiste avatar Oct 18 '16 11:10 Fablepongiste

@Fablepongiste thanks for the willingness to contribute. If it seems straight forward feel free to do a pull request. Please use quiet = getOption("pac_quiet") defaulting to FALSE as the argument. This would be in p_load and p_install and p_load. Can you talk about the need in p_delete?

@Dasonk are we missing anything that needs to be thought about or that has been missed for this feature implementation?

trinker avatar Oct 18 '16 13:10 trinker