2015-04-18
R CMD check has no ERRORs or WARNINGs, and 1 NOTE:
COPYRIGHT HOLDER: Dean Attali
2015-04-20 Prof Brian Ripley
Hmm, we see
*checking CRAN incoming feasibility ... NOTE Package has a VignetteBuilder field but no prebuilt vignette index.
Found the following (possibly) invalid URLs: URL: http://daattali.com/shiny/shinyjs-demo/ From: man/runExample.Rd man/shinyjs.Rd Status: 404 Message: Not Found
The Title field should be in title case, current version then in title case: ‘Perform Common JavaScript Operations in Shiny apps using Plain R Code’ ‘Perform Common JavaScript Operations in Shiny Apps using Plain R Code’
The Description field should not start with the package name, 'This package' or similar.
so you either did not use current R or --as-cran.
2015-04-20
Addressed previous comments:
One NOTE saying there is a possibly invalid URL, but the URL is valid, I'm not sure why the check gets a 404:
License components with restrictions and base license permitting such: MIT + file LICENSE File 'LICENSE': YEAR: 2015 COPYRIGHT HOLDER: Dean Attali Found the following (possibly) invalid URLs: URL: http://daattali.com/shiny/shinyjs-demo/ From: man/runExample.Rd man/shinyjs.Rd
Status: 404 Message: Not Found
2015-04-21 Prof Brian Ripley
Because it is not found! See the CRAN policies.
and now
2015-04-21
Ran check --as-cran with 0 ERROR, 0 WARNING, 1 NOTE
I apologize my previous submission resulted in multiple NOTES; I'm running R3.2.0 and checking --as-cran but for some reason I didn't get the same NOTES as my reviewer.
Addressed previous note:
The other NOTE I'm getting is in regards to "potentially invalid URLs" but these URLs are valid. http://daattali.com/shiny/shinyjs-demo/ and http://daattali.com/shiny/shinyjs-basic/
2015-04-21 Uwe Ligges
Then please add the final slash:
http://daattali.com/shiny/shinyjs-demo/ rather than http://daattali.com/shiny/shinyjs-demo etc.
2015-04-21
Ran check --as-cran with 0 ERROR, 0 WARNING, 1 NOTE
Addressed previous note: added final slash in URLs of web pages that were redirecting from "http://..." to "http://.../"
The only NOTE I'm getting is in regards to "potentially invalid URLs" but these URLs are valid. http://daattali.com/shiny/shinyjs-demo/ and http://daattali.com/shiny/shinyjs-basic/
2015-04-21 Prof Brian Ripley
On CRAN now
2015-05-25
R CMD check has no ERRORs or WARNINGs, and 2 NOTEs: one is about the license and one about potentially invalid URL (it's a valid URL).
2015-05-25 Prof Brian Ripley
I suppose you are talking about
Found the following (possibly) invalid URLs: URL: http://daattali.com/shiny/shinyjs-basic/ From: inst/doc/overview.html README.md Status: 404 Message: Not Found URL: http://daattali.com/shiny/shinyjs-demo/ From: man/runExample.Rd man/shinyjs.Rd inst/doc/overview.html README.md Status: 404 Message: Not Found
It seems it is misconfigured site (it does not say 'invalid'). In future comply with policies when you report!
On its way to CRAN.
2015-05-27 Prof Brian Ripley
Actually, it also fails under Roldrelease on Windows. So please fix and declare version depenndencies if needed. http://cran.r-project.org/web/checks/check_results_shinyjs.html
2015-05-27
Package accepted 2 days ago but today I got an email saying that my tests failed on windows Roldrelease. The issue is fixed
2015-05-27 Prof Brian Ripley
You also got an email about Sparc Solaris results, and that issue is not corrected:
ERROR Running the tests in ‘tests/testthat.R’ failed. Last 13 lines of output: testthat results ================================================================ OK: 4 SKIPPED: 0 FAILED: 8 1. Failure (at test-extendShinyjs.R#5): extendShinyjs throws error when gives a non-existent JS file 2. Failure (at test-extendShinyjs.R#10): extendShinyjs throws error when JS file doesn't have proper shinyjs functions 3. Failure (at test-extendShinyjs.R#15): extendShinyjs throws error when given a bad JavaScript file 4. Error: extendShinyjs finds the correct JS functions from script file 5. Failure (at test-extendShinyjs.R#26): extendShinyjs throws error when inline code doesn't have proper shinyjs functions 6. Failure (at test-extendShinyjs.R#31): extendShinyjs throws error when given bad inline code 7. Error: extendShinyjs finds the correct JS functions from inline code 8. Error: extendShinyjs finds the correct functions when both file and text are given
Error: testthat unit tests failed
Right, you did mention that, one of my functions requires V8 which is not available on very few platforms. Inside the function I do use the recommended check using requireNamespace
and if the package isn't available then I throw an error. I believe that's the correct thing to do. The tests are failing on that platform because they expect output but they get an error - and I think that's correct behaviour. The only fix I can think of is to check for the package availability inside the test suite itself -- is that right? I thought it'd be ok to leave it as-is since the tests that are failing are tests that should fail on that platform.
They should not fail if they rely on things not available on the platform.
And you are STILL not respecting
'All correspondence with CRAN must be sent to CRAN@R-project.org (not members of the team) and be in plain text ASCII (and not HTML).'
in two ways.
2015-06-23
R CMD check has no ERRORs or WARNINGs, and 2 NOTEs: one is about "possibly" invalid URLs - this is because the URLs are shiny apps on a shiny server and shiny server doesn't support returning headers, the other NOTE is about having NEWS.md
2015-06-24 Kurt Hornik
We get
Possibly mis-spelled words in DESCRIPTION:
shinyjs (11:5, 12:42)
Pls use single quotes, or write "This package".
Found the following (possibly) invalid URLs:
URL: http://daattali.com/shiny/colourInput/
From: man/colourInput.Rd
man/updateColourInput.Rd
inst/doc/overview.html
Status: 404
Message: Not Found
URL: http://daattali.com/shiny/shinyjs-basic/
From: inst/doc/overview.html
Status: 404
Message: Not Found
URL: http://daattali.com/shiny/shinyjs-demo/
From: man/runExample.Rd
man/shinyjs.Rd
inst/doc/overview.html
Status: 404
Message: Not Found
Pls fix.
Best
-k
2015-06-24
Addressed previous comment: wrapped package name in single quotes in DESCRIPTION file.
R CMD check has no ERRORs or WARNINGs, and the same 2 NOTEs: one is about having a NEWS.md file and one is about "possibly" invalid URLs. The URLs can't be fixed because they are shiny apps on a shiny server and shiny server doesn't support returning headers from curl
2015-06-24 Uwe Ligges
Thanks, on CRAN now.
2015-07-30
R CMD check has no ERRORs or WARNINGs, and 2 NOTEs: one is about "possibly" invalid URLs - this is because the URLs are shiny apps on a shiny server and shiny server doesn't support returning headers from the curl tool, the other NOTE is about having NEWS.md as a top-level file.
2015-07-30 Kurt Hornik
There's also
* checking R code for possible problems ... NOTE
formatHEXsingle: no visible global function definition for ‘colors’
formatHEXsingle: no visible binding for global variable ‘rgb’
formatHEXsingle: no visible global function definition for ‘col2rgb’
Undefined global functions or variables:
col2rgb colors rgb
Consider adding
importFrom("grDevices", "col2rgb", "colors", "rgb")
to your NAMESPACE.
Pls fix.
Re
Found the following (possibly) invalid URLs:
URL: http://daattali.com/shiny/colourInput/
From: man/colourInput.Rd
man/updateColourInput.Rd
inst/doc/overview.html
Status: 404
Message: Not Found
URL: http://daattali.com/shiny/shinyjs-basic/
From: inst/doc/overview.html
Status: 404
Message: Not Found
URL: http://daattali.com/shiny/shinyjs-demo/
From: man/runExample.Rd
man/shinyjs.Rd
inst/doc/overview.html
Status: 404
Message: Not Found
I thought that new versions of shiny would honor HEAD requests? (May
stil be devel only ...)
Best
-k
2015-07-30
2 comments were made on previous submission: 1. I had to namespace all non-base functions, 2. Shiny does not yet honour HEAD requests (there's an open pull request for it)
2015-07-30 Kurt Hornik
Thanks, on CRAN now.
I'll follow up with the shiny maintainer re HEAD.
Best -k
2015-08-12
R CMD check has no ERRORs or WARNINGs, and 2 NOTEs: one is about having NEWS.md as a top-level file and the other is the generic NOTE informing me who the maintainer of the package is.
2015-08-13 Kurt Hornik
Thanks, on CRAN now.
2015-09-05
Tested on Windows and 7 and Ubuntu 12.04. There were no ERRORs or WARNINGs and 1 NOTE that informed me who the maintainer and what the license is.
2015-09-06 Uwe Ligges
Thanks, we see:
* checking R code for possible problems ... NOTE
delay: no visible global function definition for 'runif'
oneventHelper: no visible global function definition for 'runif'
Undefined global functions or variables:
runif
Consider adding
importFrom("stats", "runif")
to your NAMESPACE.
* checking Rd files ... OK
Please fix.
2015-09-06
addressed previous comment: namespaced stats::runif
2015-09-06 Uwe Ligges
Thanks, on CRAN now.
2015-11-05
Tested on Windows and 7 and Ubuntu 12.04. There were no ERRORs or WARNINGs and 1 NOTE that informed me who the maintainer is and what the license is.
2015-11-05 Kurt Hornik
We get
Found the following (possibly) invalid URLs:
URL: http://cran.r-project.org/web/packages/shinyjs/index.html
From: inst/doc/overview.html
Status: 200
Message: OK
CRAN URL not in canonical form
The canonical URL of the CRAN page for a package is
https://cran.r-project.org/package=pkgname
Can you pls fix?
Best
-k
2015-11-05
addressed previous comment: changed URL of CRAN package to canonical form
2015-11-08 Kurt Hornik
Thanks, on CRAN now.
2015-12-30
Tested on Windows and 7 and Ubuntu 12.04. There were no ERRORs or WARNINGs and 1 NOTE that informed me who the maintainer is and what the license is.
2015-12-30 Uwe Ligges
Thanks, on CRAN now.
Although, I could not check the vignettes on Windows due to missing https support in my version of pandoc.
Best, Uwe Ligges
2016-01-21
There were no ERRORs or WARNINGs and 1 NOTE that informed me who the maintainer is and what the license is. I know that submitting twice within a month is frowned upon, but the previous submission was because I was told by a CRAN maintainer that I need to update my package to adhere to a new rule, and I want to have this package ready for the Shiny Conference next week
2016-01-21 Kurt Hornik
Thanks, on CRAN now. 2016-01-21 Kurt Hornik
Thanks, on CRAN now.
2016-03-19
R CMD check passed with no errors/warnings/notes
2016-03-19 Kurt Hornik
Thanks, on CRAN now.
2016-03-25
Tested on Windows 7 and ubuntu 14.04; no errors, warnings, or notes. Submitting updated version in response to Hadley's testthat upcoming update
2016-03-26 Kurt Hornik
We get
* checking R code for possible problems ... NOTE
closestColHex: no visible global function definition for ‘colours’
closestColHex: no visible global function definition for ‘col2rgb’
col2hex: no visible binding for global variable ‘rgb’
col2hex: no visible global function definition for ‘col2rgb’
colourPickerGadget : server: no visible global function definition for
‘colours’
createColsMap: no visible global function definition for ‘colours’
createColsMap: no visible global function definition for ‘col2rgb’
createColsMap: no visible global function definition for ‘rgb’
getLuminance: no visible global function definition for ‘col2rgb’
Undefined global functions or variables:
col2rgb colours rgb
Consider adding
importFrom("grDevices", "col2rgb", "colours", "rgb")
to your NAMESPACE file.
Can you pls fix?
2016-03-26
Fixed all the non-namespaced function calls
2016-03-27 Uwe Ligges
Thanks, on CRAN now
2016-04-24
Tested on Windows 7 and ubuntu 14.04; no errors, warnings, or notes
2016-04-24 Kurt Hornik
Thanks, on CRAN now
2016-08-21
No errors, warnings, or notes
2016-08-21 Duncan Murdoch
Thanks, on CRAN now.
2016-11-03
No errors, warnings, or notes
2016-11-03 Uwe Ligges
Thanks, on CRAN now.
2016-12-26
No errors, warnings, or notes
2016-12-26 Kurt Hornik
Thanks, on CRAN now.
Add the following code to your website.
For more information on customizing the embed code, read Embedding Snippets.