jforbes14 / eechidna

Data package containing tables, shapefiles, etc. for the 2001-2019 Australian elections and Censuses
https://jforbes14.github.io/eechidna/
43 stars 6 forks source link

pkgdown build fails #37

Open jforbes14 opened 3 years ago

jforbes14 commented 3 years ago

Hey @bryceroney could you check out the logs for the gh-pages pkgdown build? Build seems to be failing.

bryceroney commented 3 years ago

Yeah, I just saw. I wonder if the data files etc are reaching whatever memory limit there are on GitHub actions.

I'll take a look over the weekend

On Fri, 19 Feb 2021, 5:53 pm Jeremy Forbes, notifications@github.com wrote:

Hey @bryceroney https://github.com/bryceroney could you check out the logs https://github.com/jforbes14/eechidna/runs/1933283346?check_suite_focus=true for the gh-pages pkgdown build? Build seems to be failing.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/jforbes14/eechidna/issues/37, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABS2BTNVRLFGJKHIE3GBSU3S7YDHTANCNFSM4X32KXZQ .

jforbes14 commented 3 years ago

Interestingly now get this error when I try to run pkgdown::build_site()

@bryceroney when you get a chance, can you see if this error replicates when you run the above? Tried building site with previous commits and run into the same problem.

pkgdown::build_site() -- Installing package into temporary library ---------------------------------------------------------------------------- == Building pkgdown site ======================================================= Reading from: '/Users/Jeremy/Documents/R/eechidna' Writing to: '/Users/Jeremy/Documents/R/eechidna/docs' -- Initialising site ----------------------------------------------------------- -- Building home --------------------------------------------------------------- Reading 'CONDUCT.md' Reading 'CONTRIBUTING.md' -- Building function reference ------------------------------------------------- Reading 'man/aec_add_carto_f.Rd' Error : Failed to parse Rd in aec_add_carto_f.Rd ℹ unused argument (bg = bg) Error: callr subprocess failed: Failed to parse Rd in aec_add_carto_f.Rd ℹ unused argument (bg = bg)

bryceroney commented 3 years ago

I can't seem to replicate that one, have you updated all your R packages?

jforbes14 commented 3 years ago

That and adding \dontrun to some of the examples for nat_map objects seemed to do the trick. Thanks!

I can't seem to replicate that one, have you updated all your R packages?

dicook commented 1 year ago

@jforbes14 For the web site to work again:

  1. Change the setting to build the web site from master, using the docs folder (this might already be the case because the page is deploying.)
  2. Remove the GitHub actions that build to gh-pages
  3. Add link to the web site in the About section of the main repo page, https://jforbes14.github.io/eechidna/
dicook commented 1 year ago

Oh, image of hex sticker should be added to web page

jforbes14 commented 1 year ago

Done 1 & 3. Looks like the page is building now (thanks to some work by @dicook or @robjhyndman ?) and I'm not seeing any GitHub actions that build to gh-pages. Am I missing something or is this issue complete @dicook ?

dicook commented 1 year ago

On 3. The About block is on the right at the top. This is the place to add the URL.

dicook commented 1 year ago

I just removed the pkgdown.yml in the .github/workflows folder which should fix 2

dicook commented 1 year ago

You might need to go into the Actions tab @jforbes14 to remove the pkgdown workflow

jforbes14 commented 1 year ago

Ah great - About is updated.

Not seeing pkgdown workflow in the Actions tab so that looks to be deleted.

dicook commented 1 year ago

Maybe its good!