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

Checking ypr #72

Merged
merged 9 commits into from
Oct 18, 2024
Merged

Checking ypr #72

merged 9 commits into from
Oct 18, 2024

Conversation

dunkenwg
Copy link
Contributor

No description provided.

@dunkenwg
Copy link
Contributor Author

Updating the documentation gives:

ℹ Updating ypr documentation
ℹ Loading ypr
Updating vignettes
Rebuilding ypr.Rmd
Error in `purrr::map2()`:
ℹ In index: 3.
Caused by error in `map_lgl()`:
ℹ In index: 2.
Caused by error in `tools::Rd_db()`:
! installed help of package 'ypr' is corrupt
Backtrace:
 1. ypr::ypr_tabulate_parameters(ypr_population())
 2. tools::Rd_db("ypr")
      at ypr/R/tabulate-parameters.R:20:3

Two tests are also failing because the installed help of package 'ypr' is corrupt.

I've taken a look at tabulate-parameters.R, population.R, and ypr_population.Rd but haven't been able to figure anything out.

@dunkenwg dunkenwg requested a review from joethorley September 12, 2024 00:46
@joethorley
Copy link
Member

joethorley commented Sep 13, 2024

I had a quick look and can confirm issue is that the tools::Rd_db("ypr") call inside ypr_tabulate_parameters() is choking on the documentation for the ypr package. This is a nasty issue and I'm not sure how to fix.

Can you file an issue for it?

@joethorley
Copy link
Member

related to issue #74

@joethorley
Copy link
Member

@dunkenwg - has this spontaneously fixed itself?

@dunkenwg
Copy link
Contributor Author

@joethorley, we're still having the same issue locally although all of the actions are passing on GitHub

@joethorley joethorley merged commit 7338ab1 into main Oct 18, 2024
13 checks passed
@joethorley joethorley deleted the before branch October 18, 2024 00:58
@joethorley
Copy link
Member

OK I merged!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants