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

Improvements to OUP format #284

Merged
merged 15 commits into from
Aug 3, 2020
Merged

Improvements to OUP format #284

merged 15 commits into from
Aug 3, 2020

Conversation

dmkaplan2000
Copy link
Contributor

To contribute a new article template to this package, please make sure you have done the following things (note that journalname_article below is only an example name):

  • [ X] Unless you have done it in any other RStudio's projects before, please sign the individual or corporate contributor agreement for a significant pull request (it is fine not to sign it if a PR is only intended to fix a few typos). You can send the signed copy to jj@rstudio.com.

  • [ X] Add the journalname_article() function to R/article.R if the output format is simple enough, otherwise create a separate R/journalname_article.R.

  • [X ] Add the Pandoc LaTeX template inst/rmarkdown/templates/journalname_article/resources/template.tex.

  • [ X] Add a skeleton article inst/rmarkdown/templates/journalname_article/skeleton/skeleton.Rmd.

  • [X ] Add a description of the template inst/rmarkdown/templates/journalname_article/template.yaml.

  • [X ] Please include the document class file (*.cls) if needed, but please do not include standard LaTeX packages (*.sty) that can be downloaded from CTAN. Please keep the number of new files absolutely minimal, and also make examples minimal (e.g., if you need a .bib example, try to only leave one or two bibliography entries in it, and don't include one hundred items in it without using all of them).

  • [X ] Update Rd and namespace (could be done by devtools::document()).

  • [X ] Update NEWS.

  • [X ] Update README with a link to the newly supported journal.

  • Add a test to tests/testit/test-formats.R.

  • [X ] Add your name to the list of authors Authors@R in DESCRIPTION. You don't need to bump the package version in DESCRIPTION.

Lastly, please try your best to do only one thing per pull request (e.g., if you want to add two output formats, do them in two separate pull requests), and refrain from making cosmetic changes in the code base: https://yihui.name/en/2018/02/bite-sized-pull-requests/

Thank you!

David M. Kaplan and others added 13 commits January 23, 2020 00:22
Citations not working and strange things happen with a final section, but most other things seem to be working.

Template needs further work to do equations and cross-referencing correctly.
Merging back my changes accepted into rstudio version of rticles after pull request.
…box.

Modifying template to show how to use knitr::kable for generating tables
and how to use place floats at end and number lines using header-includes.
@dmkaplan2000
Copy link
Contributor Author

I have added to this pull request new changes to make OUP format by default use pandoc-citeproc for reference formatting. I would create a separate pull request for this, but I am not sure how to do that.

Also, regarding the check failure above (Travis CI), this appears to be a pandoc download issue that is unrelated to my changes. Am I supposed to do something about this?

Copy link
Member

@yihui yihui left a comment

Choose a reason for hiding this comment

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

I'm not familiar with the journal requirements, so I'll just trust you on these changes. Thanks!

@yihui yihui merged commit 25de7c1 into rstudio:master Aug 3, 2020
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 1, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants