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

R CMD CHECK fails on v2.3 #307

Closed
8 tasks done
Tracked by #297
PietrH opened this issue Sep 24, 2024 · 2 comments · Fixed by #321
Closed
8 tasks done
Tracked by #297

R CMD CHECK fails on v2.3 #307

PietrH opened this issue Sep 24, 2024 · 2 comments · Fixed by #321
Assignees
Labels
blocked documentation Improvements or additions to documentation
Milestone

Comments

@PietrH
Copy link
Member

PietrH commented Sep 24, 2024

Due to broken vignettes and examples that still use connect_to_etn():

checking R code for possible problems ... NOTE
  Found if() conditions comparing class() to string:
  Fileetn/R/list_values.R: if (class(arguments$column) == "name") ...
  Use inherits() (or maybe is()) instead.


N  checking package dependencies (5.9s)
   Package suggested but not available for checking: ‘formattable’
N  checking for future file timestamps (4m 0.7s)
   unable to verify current time
N  checking Rd line widths ... NOTE
   Rd file 'download_acoustic_dataset.Rd':
     \examples lines wider than 100 characters:
        #> * included acoustic projects:  albert, Apelafico, bpns, JJ_Belwind, leopold, MOBEIA, pc4c, SPAWNSEIS, ws2, zeeschelde
   
   These lines will be truncated in the PDF manual.
W  checking Rd \usage sections ...
   Undocumented arguments in documentation object 'get_tags'
     ‘api’
   
   Functions with \usage entries need to have the appropriate \alias
   entries, and all their arguments documented.
   The \usage entries must correspond to syntactically valid R code.
   See chapter ‘Writing R documentation files’ in the ‘Writing R
   Extensions’ manual.
E  checking examples (731ms)
   Running examples in ‘etn-Ex.R’ failed
   The error most likely occurred in:
   
   > base::assign(".ptime", proc.time(), pos = "CheckExEnv")
   > ### Name: get_acoustic_deployments
   > ### Title: Get acoustic deployment data
   > ### Aliases: get_acoustic_deployments
   > 
   > ### ** Examples
   > 
   > # Set default connection variable
   > con <- connect_to_etn()
   Error in connect_to_etn() : could not find function "connect_to_etn"
   Execution halted

❯ checking examples ... ERROR
  Running examples in ‘etn-Ex.R’ failed
  The error most likely occurred in:
  
  > base::assign(".ptime", proc.time(), pos = "CheckExEnv")
  > ### Name: get_acoustic_deployments
  > ### Title: Get acoustic deployment data
  > ### Aliases: get_acoustic_deployments
  > 
  > ### ** Examples
  > 
  > # Set default connection variable
  > con <- connect_to_etn()
  Error in connect_to_etn() : could not find function "connect_to_etn"
  Execution halted

❯ checking Rd \usage sections ... WARNING
  Undocumented arguments in documentation object 'get_tags'
    ‘api’
  
  Functions with \usage entries need to have the appropriate \alias
  entries, and all their arguments documented.
  The \usage entries must correspond to syntactically valid R code.
  See chapter ‘Writing R documentation files’ in the ‘Writing R
  Extensions’ manual.


❯ checking package dependencies ... NOTE
  Package suggested but not available for checking: ‘formattable’

❯ checking for future file timestamps ... NOTE
  unable to verify current time

❯ checking Rd line widths ... NOTE
  Rd file 'download_acoustic_dataset.Rd':
    \examples lines wider than 100 characters:
       #> * included acoustic projects:  albert, Apelafico, bpns, JJ_Belwind, leopold, MOBEIA, pc4c, SPAWNSEIS, ws2, zeeschelde
  
  These lines will be truncated in the PDF manual.

1 error ✖ | 2 warnings ✖ | 3 notes ✖
Error: R CMD check found ERRORs
Execution halted

Exited with status 1.
@PietrH PietrH mentioned this issue Sep 24, 2024
16 tasks
@PietrH PietrH changed the title pass R CMD CHECK? R CMD CHECK fails on v2.3 Sep 24, 2024
@PietrH PietrH added this to the v2.3 milestone Sep 24, 2024
@PietrH PietrH added the documentation Improvements or additions to documentation label Sep 24, 2024
PietrH added a commit that referenced this issue Sep 25, 2024
…ECK error. This function call is no longer needed.
@PietrH PietrH added the blocked label Sep 26, 2024
@PietrH
Copy link
Member Author

PietrH commented Sep 26, 2024

timestamp note is a global thing; https://stackoverflow.com/questions/63613301/r-cmd-check-note-unable-to-verify-current-time

Apparently R CMD CHECK relies on an external web resource to get the current time.

@PietrH PietrH self-assigned this Sep 26, 2024
@PietrH
Copy link
Member Author

PietrH commented Sep 30, 2024

Found new issue: #320

Decided not to let this block the beta of v2.3, but this might cause issues in the long run

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant