extendr / rextendr

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

Flesh out `principles.md` #95

Open malcolmbarrett opened 3 years ago

malcolmbarrett commented 3 years ago

94 adds principles.md to make explicit important internal coding conventions used in rextendr. That PR limited the contents of principles.md to user communication and errors. However, there are probably other conventions (or emerging conventions waiting to be codified) that would do well to be included.

Two particularly good examples of documents like this are in usethis and targets. These are both very well designed packages, and I think the principled approach to design helped get them there.

Ilia-Kosenkov commented 3 years ago

I suggest writing about to_toml() and its use cases. I strongly encourage using it every time we write a *.toml file. I believe it supports the majority of use cases that arise in Rust projects (it does not fully support TOML standard though).

malcolmbarrett commented 3 years ago

It seems like it might be good to document how to handle paths, e.g. when to use pretty_rel_path(). That's something usethis is quite explicit about in its principles.md.

clauswilke commented 3 years ago

Also, per #102, describe snapshot testing.