swcarpentry / r-novice-gapminder

R for Reproducible Scientific Analysis
http://swcarpentry.github.io/r-novice-gapminder/
Other
164 stars 538 forks source link

added alt text #807

Closed brynnelliott closed 2 years ago

brynnelliott commented 2 years ago

added alt text for https://swcarpentry.github.io/r-novice-gapminder//fig/rmd-09-ch2-sol-2.png

ErinBecker commented 2 years ago

I think the check failure was due to https://github.com/carpentries/styles/issues/641 and will be fixed with https://github.com/zkamvar/2022-10-20-workflow-bug/blob/main/update-workflows.R which will be deployed today.

zkamvar commented 2 years ago

Note: it will not be fixed until https://github.com/swcarpentry/r-novice-gapminder/pull/806 is merged, but even then, because of the way pull requests work, it will still fail after 806 is merged (sorry it's complicated).

zkamvar commented 2 years ago

Luckily, your PR is not complex and it will not break anything as long as the maintainer confirms that the syntax is correct.

naupaka commented 2 years ago

LGTM