Skip to content

Adds some more variables to arxive template#356

Merged
cderv merged 6 commits intorstudio:masterfrom
eliocamp:arxiv-includes
Dec 16, 2020
Merged

Adds some more variables to arxive template#356
cderv merged 6 commits intorstudio:masterfrom
eliocamp:arxiv-includes

Conversation

@eliocamp
Copy link
Copy Markdown
Contributor

@eliocamp eliocamp commented Dec 15, 2020 •
edited
Loading

Related to #345
The arxive template didn't have the holy

$for(header-includes)$
$header-includes$
$endfor$

it also had biblyographystyle hardcoded.

@eliocamp eliocamp changed the title Adds header-includes to arxive template Adds some customisations to arxive template Dec 15, 2020
yihui
yihui approved these changes Dec 15, 2020
View reviewed changes
Copy link
Copy Markdown
Contributor

@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 good to me. It might be worth mentioning in a NEWS bullet. Thanks!

@cderv Feel free to merge after you review it. @slemonide will need to adjust #345 accordingly.

cderv
cderv requested changes Dec 16, 2020
View reviewed changes
Copy link
Copy Markdown
Collaborator

@cderv cderv left a comment

Choose a reason for hiding this comment

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

See comment about renaming the variable below. After that it is ready to merge.

Thanks!

Comment thread NEWS.md Outdated
Comment thread inst/rmarkdown/templates/arxiv/resources/template.tex Outdated
@cderv cderv mentioned this pull request Dec 16, 2020
Closed
@eliocamp eliocamp mentioned this pull request Dec 16, 2020
Merged
@cderv cderv changed the title Adds some customisations to arxive template Adds some more variables to arxive template Dec 16, 2020
@cderv cderv merged commit 37bfb52 into rstudio:master Dec 16, 2020
@cderv
Copy link
Copy Markdown
Collaborator

cderv commented Dec 16, 2020

Thanks @eliocamp !

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Jun 21, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@cderv cderv cderv requested changes

@yihui yihui yihui 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.

3 participants

@eliocamp @cderv @yihui