search

pharmaR / riskassessment

Risk Assessment Demo App: https://rinpharma.shinyapps.io/riskassessment
https://pharmar.github.io/riskassessment/
Other
98 stars 24 forks source link

Add write-manifest.yaml #767

Closed jthompson-arcus closed 2 months ago

jthompson-arcus commented 2 months ago

@aclark02-arcus check it out. I originally opened this PR on master so that the GHA would trigger. If you look at the yaml you will notice that I had to pull in the most recent version of rsconnect for the build to work. When we update R and the PPM repo date, we should be able to pull from our snapshot.

Obviously, the first time it is run it will cause the most line changes, but check it out and let me know what you think.

aclark02-arcus commented 2 months ago

Looks good!

jthompson-arcus commented 2 months ago

I want to point out that if you look at 1a8bf23, you will see that despite the manifest being rebuilt, there is very few line changes for the manifest.json file. The changes are a result of updated a line in the file. This is showing the desired behavior or this PR, where the manifest is being built in a more standardized way.