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

enhance readme; add params to create_shinylive_url; other minor docs … #2

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

pawelru
Copy link
Contributor

@pawelru pawelru commented Sep 19, 2024

  • enhance readme:
    • add usage in vignette
    • add gifs
  • add mode and header params to create_shinylive_url()
  • other minor docs updates

@pawelru pawelru enabled auto-merge (squash) September 19, 2024 10:40
Copy link
Contributor

github-actions bot commented Sep 19, 2024

Unit Tests Summary

 1 files   2 suites   0s ⏱️
13 tests 13 ✅ 0 💤 0 ❌
50 runs  50 ✅ 0 💤 0 ❌

Results for commit f1cc09d.

♻️ This comment has been updated with latest results.

@pawelru pawelru marked this pull request as draft September 19, 2024 11:00
auto-merge was automatically disabled September 19, 2024 11:00

Pull request was converted to draft

@pawelru pawelru marked this pull request as ready for review September 19, 2024 11:01
@pawelru pawelru enabled auto-merge (squash) September 19, 2024 11:01
Copy link
Contributor

github-actions bot commented Sep 19, 2024

badge

Code Coverage Summary

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

Diff against main

Filename         Stmts    Miss  Cover
-------------  -------  ------  --------
R/parse_url.R       +3       0  +100.00%
TOTAL               +3       0  +100.00%

Results for commit: f1cc09d

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

@pawelru
Copy link
Contributor Author

pawelru commented Sep 19, 2024

I have read the CLA Document and I hereby sign the CLA

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