Hadley Wickham

Maybe `copy = "inline"` as an additional option?

Currently the header looks something like: ```R # Source: SQL [?? x 5] # Database: duckdb_connection ``` where a tibble looks like: ```R # A tibble: 53,940 × 10 ```...

Neither name really conveys connection to the database. Not sure what they should be instead, but could make use a `db_` prefix? Maybe `db_table()` and `db_simulate()`? Then could add `db_query()`...

Designing S3 generics for extensibility

1

As long as the generic contains `...`, `R CMD check` does not compare the method and generic args. See for sample package that verifies that. And R CMD check code...

Assignment outside of if

1

```R if (is.null(version_label)) { version_label

Early exits

2

If you have an if statement with a large `if` and small `else`: ```R if (cond) { # # # # # # # # } else { something() }...

Mutable state within a package

1

i.e. `the

When to use warnings instead of errors

1

One place to use a warning is where you wish you had originally made an error, but changing the behaviour now would break a lot of code.

i.e. when do you need blank lines between lines of code?