Skip to content

add no_browser hook in pre-commit - #155

Merged
donyunardi merged 1 commit into
mainfrom
43_no_browser_precommit@main
Oct 11, 2023
Merged

add no_browser hook in pre-commit#155
donyunardi merged 1 commit into
mainfrom
43_no_browser_precommit@main

Conversation

@donyunardi

Copy link
Copy Markdown
Contributor

@github-actions

Copy link
Copy Markdown
Contributor

badge

Code Coverage Summary

Filename                     Stmts    Miss  Cover    Missing
-------------------------  -------  ------  -------  ------------------------
R/include_css_js.R               7       7  0.00%    12-20
R/qenv-concat.R                 10       0  100.00%
R/qenv-constructor.R            12       0  100.00%
R/qenv-eval_code.R              49       2  95.92%   95, 104
R/qenv-get_code.R               18       0  100.00%
R/qenv-get_var.R                19       0  100.00%
R/qenv-get_warnings.R           24       0  100.00%
R/qenv-join.R                   46       0  100.00%
R/qenv-show.R                    1       1  0.00%    16
R/qenv-within.R                  7       0  100.00%
R/utils-code_dependency.R      190       7  96.32%   37, 42, 252-253, 317-320
R/utils.R                       12       0  100.00%
TOTAL                          395      17  95.70%

Diff against main

Filename      Stmts    Miss  Cover
----------  -------  ------  --------
TOTAL             0       0  +100.00%

Results for commit: 6145222

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

@github-actions

Copy link
Copy Markdown
Contributor

Unit Tests Summary

    1 files    10 suites   1s ⏱️
  92 tests   92 ✔️ 0 💤 0
186 runs  186 ✔️ 0 💤 0

Results for commit 6145222.

@vedhav vedhav left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👌🏽

@donyunardi
donyunardi merged commit bbd5298 into main Oct 11, 2023
@donyunardi
donyunardi deleted the 43_no_browser_precommit@main branch October 11, 2023 06:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants