search

iqis / Q7

Freestyle Object Oriented Programming in R
GNU General Public License v3.0
5 stars 1 forks source link

Shall `magrittr` be in the suggest list? #20

Closed elong0527 closed 2 years ago

elong0527 commented 2 years ago

It seems magrittr is not used in the function bodies. Shall it be moved to Suggests:?

https://github.com/iqis/Q7/blob/d17ee96759b6ea3c4bdc7dd9225ee1fb959ba469/DESCRIPTION#L17

iqis commented 2 years ago

I've struggled a little bit when deciding to put magrittr into Depends...

It looks like the prevailing usage of this field nowadays is limited to indicating R's version. Even for packages like dplyr which interface is designed to be almost exclusively used with with magrittr does not have it in Depends.

I suppose you're right, it should be in Suggests. It should not be in Imports, either, because Q7 itself is not written with magrittr.