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

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 = "[email protected]"),
person("Marc-Andre", "Desautels", role = c("aut", "cph"), email = "[email protected]"),
person("Dominik", "Leutnant", role = c("aut", "cph"), email = "[email protected]"),
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
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.

3 participants