Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update README.md example code #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update README.md example code #2

wants to merge 1 commit into from

Conversation

rudeboybert
Copy link

Hello, I'm thrilled to see this package! This PR adds code to your example to make it a fully-functioning reprex.

  • Explicitly creates a data frame life_df. Judging by the fact that your ggplot involved a variable lifeExp, I'm 99.7% certain this was based on gapminder data!
  • Loads the ggplot2 and bbplot packages explicitly
  • Makes the plot object name in both code chunks match.

Hello, I'm thrilled to see this package! This PR adds code to your example to make it a fully-functioning [reprex](https://www.jessemaegan.com/post/so-you-ve-been-asked-to-make-a-reprex/).

* Explicitly creates a data frame `life_df`. Judging by the fact that your `ggplot` involved a variable `lifeExp`, I'm 99.7% certain this was based on `gapminder` data!
* Loads the `ggplot2` and `bbplot` packages explicitly
* Makes the plot object name in both code chunks match.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant