search

rstudio / pins-r

Pin, discover, and share resources
https://pins.rstudio.com
Other
312 stars 63 forks source link

Move ellipsis checking #770

Closed juliasilge closed 1 year ago

juliasilge commented 1 year ago

Closes #760

This PR moves the ellipsis checking to after the early return when we don't write. The messaging now looks like this:

library(pins)
b <- board_connect()
#> Connecting to Posit Connect 2023.07.0 at <https://colorado.posit.co/rsc>

b |> pin_write(
  1:10, 
  "julia.silge/some-amazing-numbers",
  access_type = "all"
)
#> Guessing `type = 'rds'`
#> ! The hash of pin "julia.silge/some-amazing-numbers" has not changed.
#> • Your pin will not be stored.

Created on 2023-08-28 with reprex v2.0.2

github-actions[bot] commented 1 year ago

This pull request has been automatically locked. If you believe you have found a related problem, please file a new issue (with a reprex: https://reprex.tidyverse.org) and link to this issue.