Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

SAGE Journals #181

Merged
merged 4 commits into from
Jul 12, 2018
Merged

SAGE Journals #181

merged 4 commits into from
Jul 12, 2018

Conversation

oguzhanogreden
Copy link
Contributor

I added the SAGE journals.

Let me know if there's anything further to do!

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.

Looks excellent! Did you sign the contributor agreement?

@oguzhanogreden
Copy link
Contributor Author

oguzhanogreden commented Jul 12, 2018

Sorry that I didn't write in the first place that I would do it after you took a look.

I sent the email just now.

@jjallaire
Copy link
Member

@yihui Contributor agreement is signed.

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.

A few large .cls/.bst files under inst/rmarkdown/templates/sage_article/skeleton seem to be duplicated under inst/rmarkdown/templates/sage_article/resources. Could you remove the latter ones?

NEWS Outdated
@@ -1,6 +1,8 @@
rticles 0.5
---------------------------------------------------------------------

- Added the template for the SAGE Journals (thanks, @oguzhanogreden).
Copy link
Member

Choose a reason for hiding this comment

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

Please add the PR number #181.

DESCRIPTION Outdated
@@ -26,7 +26,8 @@ Authors@R: c(
person("Alessandro", "Gasparini", role = c("aut", "cph"), email = "ag475@leicester.ac.uk"),
person("Marc-Andre", "Desautels", role = c("aut", "cph"), email = "marc-andre.desautels@cstjean.qc.ca"),
person("Dominik", "Leutnant", role = c("aut", "cph"), email = "leutnant@fh-muenster.de"),
person(family = "MDPI", role = c("aut", "cph"))
person(family = "MDPI", role = c("aut", "cph")),
person("Oğuzhan", "Öğreden", role = c("ctb"), comment = c(ORCID = "0000-0002-9949-3348"))
Copy link
Member

Choose a reason for hiding this comment

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

Feel free to make yourself aut instead of ctb if you want.

@yihui yihui added this to the v0.6 milestone Jul 12, 2018
@oguzhanogreden
Copy link
Contributor Author

Done. Please note that I used inst/rmarkdown/templates/sim_article/ folder structure as a template, which also contain the duplicate of the .cls file as well.

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.

Perfect. Thanks!

I'll get rid of the duplicated .cls file in sim_article, and thanks for catching that!

@yihui yihui merged commit 9782a45 into rstudio:master Jul 12, 2018
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 2, 2020
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants