RMI-PACTA / pacta.pkgdown.rmitemplate

https://rmi-pacta.github.io/pacta.pkgdown.rmitemplate/
Other
2 stars 0 forks source link

Update all `pacta.pkgdown.template` references #19

Closed jdhoffa closed 7 months ago

jdhoffa commented 7 months ago

Relates to https://github.com/RMI-PACTA/pacta.pkgdown.rmitemplate/issues/15

cjyetman commented 7 months ago

Don't we have to update the DESCRIPTION file too? https://github.com/RMI-PACTA/pactaverse/blob/297aecb88c7d7bad6f59711d2cc108259f69d797/DESCRIPTION#L69

jdhoffa commented 7 months ago

The scope of this PR has been expanded. Updating all references now.

jdhoffa commented 7 months ago

@cjyetman I would actually expect this pkgdown action to fail at this stage, since what is on main doesn't reflect the new name of this package.

I imagine merging this will also handle https://github.com/RMI-PACTA/pacta.pkgdown.rmitemplate/issues/15#issuecomment-1918793090

cjyetman commented 7 months ago

@cjyetman I would actually expect this pkgdown action to fail at this stage, since what is on main doesn't reflect the new name of this package.

I imagine merging this will also handle #15 (comment)

How I understand it, the pkgdown GH actions do try to build the site on a PR branch, but they do not deploy (actually push to GH pages), so I would not expect them to fail (unless something is legit wrong and it would not be able to build when pushed to main).

The error I'm getting in my PRs suggests that the template package is not findable (yet?)

Caused by error: ! Could not solve package dependencies:

deps::.: Can't install dependency rmi-pacta/pacta.pkgdown.rmitemplate local::.: Can't install dependency rmi-pacta/pacta.pkgdown.rmitemplate

jdhoffa commented 7 months ago

That is what I am suggesting. The main branch of rmi-pacta/pacta.pkgdown.rmitemplate doesn't yet have an updated DESCRIPTION file, highlighting the updated package name. So when we try to install it in other actions, it can't find the package.

This PR should fix that.

cjyetman commented 7 months ago

sweet! that seems to have fixed it. rerunning pkgdown action in my PRs completes successfully now

jdhoffa commented 7 months ago

Yay

jdhoffa commented 7 months ago

New best practice: Fix things and then break them and then fix them again! And rename EVERYTHING ALWAYS