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

Added Biometrics Template #170

Merged
merged 15 commits into from
Oct 9, 2018
Merged

Added Biometrics Template #170

merged 15 commits into from
Oct 9, 2018

Conversation

daltonhance
Copy link
Contributor

This creates a template for a submission to Biometrics based off the journal LaTeX class. There is still some housekeeping in terms of creating a helpfile and exporting the namespace, but I've tested it on my machine.

Apologies for any poor Github etiquette, this is my first external pull request.

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.

  1. Please rebase against the current master branch, and force push.

  2. Please add

Thanks!

@yihui yihui force-pushed the master branch 3 times, most recently from 8023cbf to 928d6cd Compare July 3, 2018 16:11
daltonhance added a commit to daltonhance/rticles that referenced this pull request Jul 23, 2018
@daltonhance
Copy link
Contributor Author

My fork has been rebased and force pushed. I have added myself to the list of authors, update README.md, and added a test.

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.

Excellent. With two possible minor changes, I think we are ready to go!

BTW, did you sign the contributor agreement? https://github.com/rstudio/rticles/blob/master/.github/PULL_REQUEST_TEMPLATE.md

NEWS Outdated
@@ -3,6 +3,8 @@ rticles 0.6 (unreleased)

- Added the template for the SAGE Journals (thanks, @oguzhanogreden, #181).

- Added the template for Biometrics (@daltonhance, #170)
Copy link
Member

Choose a reason for hiding this comment

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

Could you also add an entry in README.md?

#' }
#'
#' @export
biometrics_article <- function(..., keep_tex = TRUE, citation_package = 'none'){
Copy link
Member

Choose a reason for hiding this comment

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

Are you sure you don't want to set citation_package = 'natbib' as the default?

@daltonhance
Copy link
Contributor Author

I just sent my contributor agreement off. I changed the default citation_package to natbib. I did add a link to the Biometrics journal on the README.md on my fork.

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.

Great! We can merge this PR once @jjallaire confirms on the CLA. Thank you very much!

@yihui yihui merged commit 0650a85 into rstudio:master Oct 9, 2018
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 2, 2020
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.

2 participants