Skip to content

Fix test for dev dbplyr - #31

Open
mgirlich wants to merge 1 commit into
hoxo-m:masterfrom
mgirlich:dev-dbplyr-compat
Open

Fix test for dev dbplyr#31
mgirlich wants to merge 1 commit into
hoxo-m:masterfrom
mgirlich:dev-dbplyr-compat

Conversation

@mgirlich

Copy link
Copy Markdown

The new dbplyr version slightly changes the generated SQL. I adapted the test so that it works with the new dbplyr version.
It might be worth changing this to a snapshot test as relying on the exact translation may be a bit brittle.

@hadley

hadley commented May 20, 2022

Copy link
Copy Markdown

We're planning to release dbplyr to CRAN on June 3.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants