Owen Jones
Owen Jones
Closes https://github.com/cynkra/dm/issues/1533. (I opened that issue a moment ago, accidentally from a different account!)
### Error description If a module imports, and then re-exports, an object from another module, then `box::help()` doesn't look for documentation in the right place. We can see this with...
**Is your feature request related to a problem? Please describe.** R developers often writes documentation in the form of "roxygen comments", in the same file as their code. If the...
An attempt to add support for [Amazon Redshift](https://aws.amazon.com/redshift/) connections. Redshift is based on Postgres - which is great, because {dm} already has excellent Postgres support :grin: So for the most...
Fixes #2213. I had to move the "smart default" logic for `.names` up one level, from `get_src_tbl_names()` into `dm_from_con()` - but I think that is a better place to keep...
A new `.names` parameter was added to `dm_from_con()` in v1.0.6 (https://github.com/cynkra/dm/pull/1790) - but it's currently ignored when `learn_keys = TRUE` and the subsequent key-querying is successful. That's because `dm_from_con()` [returns...
As discussed in #562. Things to note: * Since `roxygen2::parse_file()` [does actually parse the file](https://github.com/r-lib/roxygen2/blob/c03a6711eea3f7dc16e55f0ab0babac9ff40d9ea/R/tokenize.R#L21), I think we have to allow this hook to fail in two ways: - If...
We can specify a new parameter with `new_quant_param()` or `new_qual_param()`. The docs state that `label` is an optional parameter: https://github.com/tidymodels/dials/blob/210055fa7f4e9483567fbf3bd642c31b5116608e/R/constructors.R#L34-L36 But if no label is specified, we get an opaque...
The current regex implies that we should have something like: ```hcl data "aws_iam_policy_document" "example" { statement { actions = ["lambda:InvokeFunction"] resources = [ aws_lambda_function.example.arn, aws_lambda_function.example.arn:* #
Set an explicit lowercase name for the summary column, to avoid problems with Redshift case conversion. Fixes #1601. We could use any other lowercase name here, if `"max"` is likely...