Skip to content

fix: make Connect dependencies reproducible - #111

Open
tdstein wants to merge 1 commit into
mainfrom
fix/reproducible-connect-dependencies
Open

fix: make Connect dependencies reproducible#111
tdstein wants to merge 1 commit into
mainfrom
fix/reproducible-connect-dependencies

Conversation

@tdstein

@tdstein tdstein commented Aug 14, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Add a committed renv lockfile and pin the compatible Ellmer/Shinychat revisions.
  • Restore the locked environment in CI and generate the Connect manifest from the exact commit being deployed.
  • Remove the stale checked-in manifest and limit the generated bundle to production app files.

@github-actions

Copy link
Copy Markdown

Preview deployed to Connect (dogfood.team.pct.posit.it): https://dogfood.team.pct.posit.it/connect/#/apps/d7a36cae-8f27-448b-a478-61b81fbe3942/draft/361527

Deployed from commit 475f914.

@github-actions

Copy link
Copy Markdown

Preview deployed to Connect (connect.staging.pct.posit.it): https://connect.staging.pct.posit.it/connect/#/apps/ad662e1b-5048-4acc-9ad7-f9478c92274e/draft/1798

Deployed from commit 475f914.

@tdstein
tdstein requested a review from simonpcouch August 14, 2026 17:45
@tdstein

tdstein commented Aug 14, 2026

Copy link
Copy Markdown
Collaborator Author

@simonpcouch, I don't know the R ecosystem very well, so I'm not confident that this is the correct approach. The problem I'm trying to solve is that the original manifest.json kept getting out of date. This change works, but also slows down CI quite a bit.

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.

1 participant