Skip to content

Cohort_2_Chapters_0_1_2 - #10

Merged
jonthegeek merged 2 commits into
r4ds:mainfrom
dsollberger:cohort_2_start
Jul 1, 2026
Merged

jonthegeek merged 2 commits into
r4ds:mainfrom
dsollberger:cohort_2_start

Conversation

@dsollberger

Copy link
Copy Markdown
Contributor
  • revised first three sections to align with second edition to textbook
  • streamlined main terminology
  • pruned discussion (could still elaborate in meetings)

* streamlined main terminology
* pruned discussion (could still elaborate in meetings)

@jonthegeek jonthegeek left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I capitalized some bullets to match the style of the rest of the intro, otherwise this looks great. I'm going to accept my suggestion then merge, but I wanted to share the review. Thanks for handling this!

Comment thread index.Rmd Outdated
Comment thread 01_designing-research.Rmd

@jonthegeek jonthegeek left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Ok, approving now that my capitalization is merged. Hopefully this will deploy without issues!

@jonthegeek
jonthegeek enabled auto-merge (squash) June 29, 2026 10:28
@jonthegeek
jonthegeek merged commit 18bb578 into r4ds:main Jul 1, 2026
1 check passed
@dsollberger
dsollberger deleted the cohort_2_start branch July 3, 2026 20:08
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