Skip to content

Update pkgdown GitHub Action #4151

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

Merged
merged 4 commits into from
Jul 29, 2020

Conversation

yutannihilation
Copy link
Member

@yutannihilation yutannihilation commented Jul 29, 2020

pkgdown action now fails with the following error. It seems we now need to configure username and email explicitly (c.f. r-lib/actions#135). Since the latest pkgdown.yml already handles this and it contains some other improvements, I copied it and added some tweaks (including publish event, installing binary packages, and installing tidytemplate).

https://github.com/tidyverse/ggplot2/runs/921163458#step:7:520:

── Commiting updated site ──────────────────────────────────────────────────────
Running git add -A .
Running git commit --allow-empty -m \
  'Built site for ggplot2: 3.3.2.9000@8f7b03c'

*** Please tell me who you are.

Run

  git config --global user.email "[email protected]"
  git config --global user.name "Your Name"

to set your account's default identity.
Omit --global to set the identity only in this repository.

fatal: unable to auto-detect email address (got 'runner@Mac-1595979566438.(none)')

Error: System command 'git' failed, exit status: 128, stdout & stderr were printed

@yutannihilation
Copy link
Member Author

@karawoo
Could you approve this as well? Sorry, I failed to address the pkgdown failure in #4147...
Confirmed this works on my forked repo:
https://github.com/yutannihilation/ggplot2/runs/921446697?check_suite_focus=true

@karawoo karawoo merged commit b4bc293 into tidyverse:master Jul 29, 2020
@yutannihilation
Copy link
Member Author

Thanks!

@yutannihilation yutannihilation deleted the ci/fix-pkgdown-action branch July 29, 2020 05:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants