search

rstudio / pins-r

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

Simplify Connect config for testing #709

Closed juliasilge closed 1 year ago

juliasilge commented 1 year ago

I don't think we need Python enabled on Connect for these tests: https://github.com/rstudio/pins-r/blob/90b68ba6199f695b785b64d5312385727c51a2d9/.github/rstudio-connect.gcfg#L15-L17 Even if we eventually run a roundtrip test (R 🔄 Python) we don't need Python installed on Connect, just the runner.

We could test on latest instead of pinning to a specific version of Connect: https://github.com/rstudio/pins-r/blob/90b68ba6199f695b785b64d5312385727c51a2d9/.github/workflows/posit-connect.yaml#L14

juliasilge commented 1 year ago

Sounds like testing on latest is recommended by the Connect team.

github-actions[bot] commented 1 year ago

This issue 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.