tar_prql shorthand?

Open daranzolin opened this issue 1 year ago • 2 comments

Apr 13 '24 15:04 daranzolin

Rather than a separate function, this could maybe better work as a separate argument to tar_sql e.g. is_prql = TRUE. Then call prqlr::prql_compile() within source_sql_to_dataframe.

Apr 14 '24 23:04 daranzolin

There are several options for implementation.

Option 1: PRQL in .sql files (or .prql files)

Pros: Minimal changes needed to API and code. Cons: Can't modify and preview query results in RStudio (but could in Visual Studio), no syntax highlighting

Option 2: PRQL in .qmd files

Pros: Can modify and view query results in RStudio. Cons: Significant changes to underlying code, plus quarto dependency, plus some visual awkwardness:

...
tar_prql(query, "queries/prql_query.qmd")
...

Option 3: Wait until RStudio supports .prql files

Enhancement suggested in https://github.com/rstudio/rstudio/issues/14547 Pros: Minimal changes needed. Cons: Might never happen

May 03 '24 19:05 daranzolin