Skip to content

Commit

Permalink
Fix vignette problem
Browse files Browse the repository at this point in the history
  • Loading branch information
yannikbuhl committed Jun 29, 2019
1 parent d2be771 commit 020a41e
Show file tree
Hide file tree
Showing 24 changed files with 257 additions and 6 deletions.
2 changes: 1 addition & 1 deletion .Rbuildignore
Original file line number Diff line number Diff line change
Expand Up @@ -9,4 +9,4 @@
^README-.*\.png$
^cran-comments\.md$
^newsanchor\.png$
^vignettes\/scrape-nyt_cache\/*$
^examples$
2 changes: 2 additions & 0 deletions CRAN-RELEASE
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
This package was submitted to CRAN on 2019-03-05.
Once it is accepted, delete this file and tag the release (commit 9879502857).
2 changes: 0 additions & 2 deletions DESCRIPTION
Original file line number Diff line number Diff line change
Expand Up @@ -20,8 +20,6 @@ Imports:
xml2,
lubridate,
askpass
Remotes:
jeroen/askpass
Suggests:
dplyr,
knitr,
Expand Down
1 change: 1 addition & 0 deletions NEWS.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,3 +5,4 @@
# version 0.1.1 (2019-06-25)

- fixed issue with the NYT vignette (now suggests package "textdata")
- moved vignette to 'example' folder in root directory
4 changes: 3 additions & 1 deletion cran-comments.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,4 +11,6 @@ There are currently no downstream dependencies for this package.

## Fixed: Version 0.1.1
Version 0.1.1 contains a fix for one of the vignettes which had seen a change
in package dependencies. DESCRIPTION now suggests to load textdata as well.
in package dependencies. DESCRIPTION now suggests to load textdata as well
and vignette (which needs to be run interactively) is now in an examples
folder which is included in .Rbuildignore.
File renamed without changes.
File renamed without changes.
File renamed without changes.
10 changes: 10 additions & 0 deletions examples/scrape-nyt_cache/html/__packages
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
base
newsanchor
robotstxt
httr
xml2
rvest
dplyr
stringr
tidytext
textdata
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
239 changes: 239 additions & 0 deletions examples/scrape-nyt_files/figure-html/plot-results-1.svg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
1 change: 0 additions & 1 deletion tests/testthat.R
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
library(testthat)
library(newsanchor)
library(askpass)
library(mockery)
test_check("newsanchor")
2 changes: 1 addition & 1 deletion tests/testthat/test_get_everything.R
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,7 @@ testthat::test_that("Test that function stops when invalid date format is provid
# INVALID API KEY --------------------------------------------------------------------
testthat::test_that("test that function raises warning if API key invalid.", {
testthat::expect_warning(newsanchor::get_everything(api_key = "thisisnotanapikey",
query = "sports"),
query = "sports"),
regexp = "The search resulted in the following error message:")
})

Expand Down

0 comments on commit 020a41e

Please sign in to comment.