Dark Mode

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Adding INFORMS template#460

Merged
cderv merged 20 commits intorstudio:masterfrom
robjhyndman:master
Jan 3, 2022
Merged

Adding INFORMS template#460
cderv merged 20 commits intorstudio:masterfrom
robjhyndman:master

Conversation

Copy link
Contributor

robjhyndman commented Dec 23, 2021

How to contribute a new output format ?

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] This project uses a Contributor Licence Agreement (CLA) that you'll be asked to sign when opening a PR. This is required for a significant pull request (it is fine not to sign it if a PR is only intended to fix a few typos). We use a tool called CLA assistant for that.
    You could also, unless you have done it in any other RStudio's projects before, sign the individual or corporate contributor agreement. 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 ] Document your function using roxygen2. Markdown syntax is supported. Refer to https://roxygen2.r-lib.org/articles/rd-formatting.html for formatting references.

  • [x ] Add the Pandoc LaTeX template inst/rmarkdown/templates/journalname/resources/template.tex.

  • [x ] Add a skeleton article inst/rmarkdown/templates/journalname/skeleton/skeleton.Rmd.

  • [x ] Add a description of the template inst/rmarkdown/templates/journalname/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. If you are using TinyTeX or TeX Live, you can verify if a package is available on CTAN via tinytex::parse_packages(files = "FILENAME"") (e.g., when FILENAME is plain.bst, it should return "bibtex", which means this file is from a standard CTAN package). Please keep the number of new files absolutely minimal (e.g., do not include PDF output files), 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 too many 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. Please add your Github username and the full name of the journal (follow other examples in the list).

  • [x ] Add a test to tests/testit/test-formats.R by adding a line test_format("journalname"). We try to keep them in alphabetical order.

  • [ 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!

Copy link

CLAassistant commented Dec 23, 2021 *
edited
Loading


All committers have signed the CLA.

robjhyndman mentioned this pull request Dec 23, 2021
3 tasks
cderv requested changes Dec 23, 2021
Copy link
Collaborator

cderv left a comment

Choose a reason for hiding this comment

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

Thanks a lot !

A few comments below.

Also I see this in the output

It does not seem to come from the template. Is this expected in all produced articles ?
I am just asking out of curiosity.

Copy link
Contributor Author

robjhyndman commented Dec 24, 2021

All issues addressed in the latest commit a225596

The red text in the output is hard-coded into the informs.cls file.

robjhyndman added 2 commits December 24, 2021 15:16
cderv requested changes Dec 24, 2021
Copy link
Collaborator

cderv left a comment

Choose a reason for hiding this comment

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

Just a last comment that I could include before merging - we need a requirement on Pandoc version due to usage of special feature in the template.

Also, can you put the change related to IOP article work in another PR ? Another branch from master ?

Thank you.

cderv approved these changes Jan 3, 2022
Copy link
Collaborator

cderv left a comment

Choose a reason for hiding this comment

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

Thanks !

I'll make a few more tweaks then merge !

And Happy new year @robjhyndman !

cderv merged commit 9cfda82 into rstudio:master Jan 3, 2022
cderv linked an issue Feb 11, 2022 that may be closed by this pull request
3 tasks
github-actions bot locked as resolved and limited conversation to collaborators Jul 4, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

cderv cderv 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.

Add INFORMS styles

3 participants