Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix 1031 #1063

Closed

Conversation

ilyaZar
Copy link
Contributor

@ilyaZar ilyaZar commented Jul 5, 2023

Fix #1031

Just to make sure I do not miss anything @jdavis-EliLilly , is it just this one-liner here?

## Deploy to Posit Connect or ShinyApps.io ----
## Add/update manifest file (optional; for Git backed deployment on Posit )
rsconnect::writeManifest()

From the documentation I understand that rsconnect::writeManifest() is for repeated usage, similar to rsconnect::deployApp() for repeated deployment (whereas the other functions in the file are usually called once for file creation only). So I would think this is a reasonable place for the rsconnect::writeManifest() call, or?

- add chunk inside R/03_deploy.R that has the relevant call to
  rsconnect::writeManifest
- while already here: update file with RStudio -> Posit
- some styling (--- for code folding and re-group function calls a bit)
- consistent code formatting with ``
- some typos
- remove some redundant whitespace
- Remove double "####" headers in a row
- update with rsconnect::writeManifest() information
@ColinFay ColinFay changed the base branch from dev to temp-branch-for-cherry-pick-1063 October 11, 2023 08:14
@ColinFay
Copy link
Member

I've cherry-picked and merged to https://github.com/ThinkR-open/golem/tree/1063-cherry-picked

@ColinFay ColinFay closed this Oct 11, 2023
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