Skip to content

Commit

Permalink
rjournal_web_article() doesn't produce pdf - use rjournal_article()
Browse files Browse the repository at this point in the history
Ref #112
  • Loading branch information
mitchelloharawild committed Oct 6, 2023
1 parent aca4311 commit 10c5a5e
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 1 deletion.
6 changes: 6 additions & 0 deletions NEWS.md
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,12 @@
* handling of longtable
* utilise system sty file, not author supplied

Breaking changes
* `rjournal_web_article()` will no longer produce the PDF via dual rendering,
to produce both HTML and PDF please use `rjournal_article()`. This breaking
change has been done to improve semantics and allow a way to quickly work
on articles by specifying which output format (web or pdf) is desired.

# rjtools 1.0.11

* removed return value, and set create_article to write a message
Expand Down
8 changes: 7 additions & 1 deletion R/rj_web_article.R
Original file line number Diff line number Diff line change
@@ -1,3 +1,9 @@
#' @export
#' @rdname rjournal_article
rjournal_web_article <- rjournal_article
rjournal_web_article <- function(toc = FALSE, self_contained = FALSE, ...) {
base_format <- rjournal_article(
self_contained = self_contained, toc = toc, ...
)
base_format$on_exit <- function() {}
base_format
}

0 comments on commit 10c5a5e

Please sign in to comment.