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

Update man page for git_vaccinate() #1867

Merged
merged 5 commits into from
Jun 23, 2023
Merged

Update man page for git_vaccinate() #1867

merged 5 commits into from
Jun 23, 2023

Conversation

jakobarendt
Copy link
Contributor

I updated the man page for the git_vaccinate() command such that it documents all of the adjustments made to the .gitignore. I also reordered the list of files and directories that the command adds to the .gitignore such that it fits the order in the output returned after calling the command.

Sorry if my line breaks do not follow your standard conventions. I was a little unsure about that.

I updated the man page for the git_vaccinate() command such that it documents **all** of the adjustments made to the `.gitignore`. I also reordered the list of files and directories that the command adds to the `.gitignore` such that it fits the order in the output returned after calling the command.

Sorry if my line breaks do not follow your standard conventions. I was a little unsure about that.
@jennybc
Copy link
Member

jennybc commented Jun 22, 2023

Can you implement this using markdown syntax, i.e. the backticks that were previously there, as opposed to \code{}? And also run devtools::document() to regenerate the .Rd file.

Update local branch to the intermittent changes in the upstream GitHub
repo.
I corrected my updates suggested in the commit 839512c according to the comments from @jennybc. Thank you!
@jakobarendt
Copy link
Contributor Author

Done. Thanks for the feedback!

@jennybc
Copy link
Member

jennybc commented Jun 23, 2023

Thanks!

@jennybc jennybc merged commit 2100af9 into r-lib:main Jun 23, 2023
13 checks passed
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