You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
First off, thanks for this package!! It's short and sweet and does exactly what we need. We have a couple of packages that help provide and organize spatial metadata associated with environmental monitoring:
Our packages need to be able to work well for both analysts sitting in front of RStudio and also inside of batch scripts. For batch scripts we typically ignore messages intended for a human user. This is easy to do if underlying code consistently uses message(), warning() and stop(). However, for any packages that print() or cat() we have to write specialized code using capture.output() in order to run things in verbose = FALSE mode.
I think it would be a general improvement if you swapped out all use of print() for message().
The text was updated successfully, but these errors were encountered:
First off, thanks for this package!! It's short and sweet and does exactly what we need. We have a couple of packages that help provide and organize spatial metadata associated with environmental monitoring:
Our packages need to be able to work well for both analysts sitting in front of RStudio and also inside of batch scripts. For batch scripts we typically ignore messages intended for a human user. This is easy to do if underlying code consistently uses
message()
,warning()
andstop()
. However, for any packages thatprint()
orcat()
we have to write specialized code usingcapture.output()
in order to run things inverbose = FALSE
mode.I think it would be a general improvement if you swapped out all use of
print()
formessage()
.The text was updated successfully, but these errors were encountered: