[llrs-roche]

Pull Request

Fixes #292

I keep mentions to insightsengineering on configuration files that need it

[github-actions]

Code Coverage Summary

Filename                         Stmts    Miss  Cover    Missing
-----------------------------  -------  ------  -------  -------------
R/qenv-c.R                          55       0  100.00%
R/qenv-class.R                      13       0  100.00%
R/qenv-concat.R                      7       0  100.00%
R/qenv-constructor.R                 1       0  100.00%
R/qenv-errors.R                      4       4  0.00%    6-9
R/qenv-eval_code.R                  63       1  98.41%   40
R/qenv-extract.R                    30       0  100.00%
R/qenv-get_code.R                   24       0  100.00%
R/qenv-get_env.R                     3       1  66.67%   27
R/qenv-get_messages.r                5       0  100.00%
R/qenv-get_outputs.R                 6       0  100.00%
R/qenv-get_var.R                    13       1  92.31%   13
R/qenv-get_warnings.R                5       0  100.00%
R/qenv-join.R                        1       1  0.00%    13
R/qenv-length.R                      2       1  50.00%   2
R/qenv-show.R                       29      29  0.00%    19-50
R/qenv-within.R                      8       0  100.00%
R/utils-get_code_dependency.R      258       3  98.84%   160, 258, 326
R/utils.R                           42       0  100.00%
TOTAL                              569      41  92.79%

Diff against main

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

Results for commit: b9f54a6

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

[github-actions]

Unit Tests Summary

  1 files   14 suites   5s ⏱️
190 tests 186 ✅ 4 💤 0 ❌
274 runs  270 ✅ 4 💤 0 ❌

Results for commit b9f54a6.

♻️ This comment has been updated with latest results.

[osenan]

I think that there are some more lines that should be changed.
In addition look at files:

*inst/_pkgdown.yml

• many files within the github workflow:
  • check.yml
  • cla.yml
  • release.yml
  • post-release.yml
  • scheduled.yml
• then for the issue template:
  • cran-release.yml
  • release.yml

In other PRs many of these files have been modified, they still have references to insights engineering

[llrs-roche]

I think that there are some more lines that should be changed. In addition look at files:

• inst/_pkgdown.yml

I think this is a leftover from some internal build that can be removed because the one used by pkgdown is on the top level directory of the package/repo.

• many files within the github workflow:

  • check.yml
  • cla.yml
  • release.yml
  • post-release.yml
  • scheduled.yml

• then for the issue template:

  • cran-release.yml
  • release.yml

In other PRs many of these files have been modified, they still have references to insights engineering

In other PR I have modified and reverted it back. Many if not all don't need changes as the CI tooling is still on insightsengineering and that won't change for now

[osenan]

Thank you for answering my comments. I consider the answers good to approve the PR, only the reference to contributing is pending and it can be discussed without blocking the approval.

[github-actions]

Unit Test Performance Difference

Test Suite	$Status$	Time on main	$±Time$	$±Tests$	$±Skipped$	$±Failures$	$±Errors$
qenv_get_code	💚	$3.01$	$-1.10$	$0$	$0$	$0$	$0$

Results for commit bd04c98

♻️ This comment has been updated with latest results.