search

extendr / rextendr

An R package that helps scaffolding extendr-enabled packages or compiling Rust code dynamically
https://extendr.github.io/rextendr/
Other
184 stars 27 forks source link

Remove {devtools} dependency #90

Closed clauswilke closed 3 years ago

clauswilke commented 3 years ago

We should remove {devtools} from imports. It's a pretty heavy dependency, and we use it only to call {roxygen2}. We could do that directly.

See also: https://github.com/extendr/rextendr/issues/88#issuecomment-802625576

clauswilke commented 3 years ago

(We will need to list {devtools} in supports, then, but that's fine.}

yutannihilation commented 3 years ago

What about moving {devtools} to Suggest dependency and making rextendr::document() abort when {devtools} is not installed?

clauswilke commented 3 years ago

That works for me also.