From e2216eb93011956f93e7941143c028d9ee6b1442 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Llu=C3=ADs=20Revilla?= <185338939+llrs-roche@users.noreply.github.com> Date: Wed, 29 Jan 2025 17:04:27 +0100 Subject: [PATCH] Add a new regexp to the r-cmd-check-note blocklist (#291) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This PR is to add the notes to the CI part per https://github.com/insightsengineering/teal.reporter/issues/288 but only for teal.reporter. The changes do not look to the general issue as vignettes failing to build could be due to a number of reasons. As far as I know the string I added is specific for this error. To fix the issue PR #290 was already merge so I don't expect any failures now. This could be added to the default/general check from r.pkg.template if it meets the IDR requirements. --------- Signed-off-by: Lluís Revilla <185338939+llrs-roche@users.noreply.github.com> Co-authored-by: Marcin <133694481+m7pr@users.noreply.github.com> --- .github/workflows/check.yaml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/.github/workflows/check.yaml b/.github/workflows/check.yaml index 0bc539c3..5b966726 100644 --- a/.github/workflows/check.yaml +++ b/.github/workflows/check.yaml @@ -39,6 +39,7 @@ jobs: checking Rd .usage sections .* NOTE checking for unstated dependencies in vignettes .* NOTE checking top-level files .* NOTE + Files named as vignettes but with no recognized vignette engine.* NOTE unit-test-report-brand: >- https://raw.githubusercontent.com/insightsengineering/hex-stickers/main/thumbs/teal.png deps-installation-method: setup-r-dependencies @@ -68,6 +69,7 @@ jobs: checking Rd .usage sections .* NOTE checking for unstated dependencies in vignettes .* NOTE checking top-level files .* NOTE + Files named as vignettes but with no recognized vignette engine.* NOTE deps-installation-method: setup-r-dependencies lookup-refs: | insightsengineering/rlistings