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.

Sign up for GitHub

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

Jump to bottom

Set up sections properly. #116

Merged
merged 1 commit into from
Jan 21, 2024
Merged

Set up sections properly. #116

merged 1 commit into from
Jan 21, 2024

Conversation

jonthegeek
Copy link
Member

Sometimes it helps to read the book! https://bookdown.org/yihui/bookdown/markdown-extensions-by-bookdown.html#special-headers

This gets rid of the warning about more RMDs than chapter headers.

@jonthegeek
Set up sections properly.
0ff1252 
Sometimes it helps to read the book! https://bookdown.org/yihui/bookdown/markdown-extensions-by-bookdown.html#special-headers

This gets rid of the warning about more RMDs than chapter headers.
@jonthegeek
Copy link
Member Author

@Ken-Vu @lgibson7 After this is accepted, you'll need to updated any open PRs to get the changes. You can do that in the github UI if the PR has been pr_push()ed, or by running usethis::pr_merge_main() in your in-progress PR in RStudio.

lgibson7
lgibson7 approved these changes Jan 21, 2024
View reviewed changes
Copy link
Member

@lgibson7 lgibson7 left a comment

Choose a reason for hiding this comment

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

Looks good. Thanks @jonthegeek !

@lgibson7 lgibson7 merged commit b0230d7 into main Jan 21, 2024
1 check passed
@lgibson7 lgibson7 deleted the cleaner-parts branch January 21, 2024 00:34
@lgibson7
Copy link
Member

@Ken-Vu for reference, this is how you update a PR that's open
Screenshot 2024-01-20 at 4 37 27 PM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lgibson7 lgibson7 lgibson7 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jonthegeek @lgibson7