search

insightsengineering / roxy.shinylive

roxygen2 extensions for shinylive
https://insightsengineering.github.io/roxy.shinylive/
Other
6 stars 2 forks source link

Initialize the package #1

Closed pawelru closed 2 months ago

github-actions[bot] commented 3 months ago

badge

Code Coverage Summary

Filename                     Stmts    Miss  Cover    Missing
-------------------------  -------  ------  -------  ---------
R/parse_url.R                    9       0  100.00%
R/tag_examplesShinylive.R       72       0  100.00%
TOTAL                           81       0  100.00%

Results for commit: 0b2a818a25bac8333bf52082b29e96f626196155

Minimum allowed coverage is 80%

:recycle: This comment has been updated with latest results

github-actions[bot] commented 3 months ago

Unit Tests Summary

 1 files   2 suites   0s :stopwatch: 13 tests 13 :white_check_mark: 0 :zzz: 0 :x: 50 runs  50 :white_check_mark: 0 :zzz: 0 :x:

Results for commit 0b2a818a.

:recycle: This comment has been updated with latest results.

gogonzo commented 3 months ago

How does it work? Do I understand correctly that at https://shinylive.io/r/app/ there is simple server which decodes an URI code= parameter and returns a webR app? Looks almost serverless, except a simple call to the shinylive

pawelru commented 3 months ago

How does it work? Do I understand correctly that at https://shinylive.io/r/app/ there is simple server which decodes an URI code= parameter and returns a webR app? Looks almost serverless, except a simple call to the shinylive

That's right. It's quite simple. That's why I thought we can easily integrate it to the docs

donyunardi commented 2 months ago

Forgot to mention that I can't pass the R CMD check locally

image

However, the CI able to run the check successfully. Is it the version of my tidy cli?

pawelru commented 2 months ago

Forgot to mention that I can't pass the R CMD check locally

I'm a little bit puzzled by what tidy returns. It looks like yours does not like allowfullscreen and mine does not like allow="fullscreen". I think I'm going to remove it completely to limit this dependency. If really needed (which I have doubts) we can bring this back