Skip to content
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

[IN PROGRESS] Update to pivot functions #91

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

mine-cetinkaya-rundel
Copy link
Member

@mine-cetinkaya-rundel mine-cetinkaya-rundel commented Mar 2, 2020

What I've already done:

  • spread/gather -> pivot_*
  • Fix a few typos
  • Add missing links to other primers
  • Update one slightly confusing MC question

What we need to make a decision on before PR can be merged:

  • There was one note in the existing primer "[To be replaced with a video]". I haven't taken that out. We can make the video or take out that note.
  • There are two existing videos (search for "vimeo" in the document) that will need to be updated. I imagine @garrettgman has the keynote slides that go with them. I'd be happy to update if you can share the Keynote slides, or we can hold off until you can make the videos and merge the PR then.

I'll leave the PR title "in progress" until we make a decision on these.

- spread/gather -> pivot_*
- Fix a few typos
- Add missing links to other primers
- Update one slightly confusing MC question
@mine-cetinkaya-rundel mine-cetinkaya-rundel requested review from apreshill and removed request for garrettgman March 2, 2020 15:25
Copy link

@apreshill apreshill left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, I left some editorial comments (feel free to use what is useful, ignore what it not), and I think I only noticed one typo :)

tidy-data/01-Reshape-Data/01-Reshape-Data.Rmd Outdated Show resolved Hide resolved
tidy-data/01-Reshape-Data/01-Reshape-Data.Rmd Outdated Show resolved Hide resolved
tidy-data/01-Reshape-Data/01-Reshape-Data.Rmd Outdated Show resolved Hide resolved
tidy-data/01-Reshape-Data/01-Reshape-Data.Rmd Show resolved Hide resolved
tidy-data/01-Reshape-Data/01-Reshape-Data.Rmd Outdated Show resolved Hide resolved
tidy-data/01-Reshape-Data/01-Reshape-Data.Rmd Outdated Show resolved Hide resolved
tidy-data/01-Reshape-Data/01-Reshape-Data.Rmd Outdated Show resolved Hide resolved
tidy-data/01-Reshape-Data/01-Reshape-Data.Rmd Show resolved Hide resolved
tidy-data/01-Reshape-Data/01-Reshape-Data.Rmd Outdated Show resolved Hide resolved
tidy-data/01-Reshape-Data/01-Reshape-Data.Rmd Show resolved Hide resolved
andrewheiss added a commit to andrewheiss/primers that referenced this pull request Jan 3, 2024
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