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

Fix check notes #454

Merged
merged 3 commits into from
Nov 19, 2024
Merged

Fix check notes #454

merged 3 commits into from
Nov 19, 2024

Conversation

seabbs
Copy link
Contributor

@seabbs seabbs commented Nov 19, 2024

Description

As the title this PR closes easily resolved check notes.

Checklist

  • My PR is based on a package issue and I have explicitly linked it.
  • I have included the target issue or issues in the PR title in the for Issue(s) issue-numbers: PR title
  • I have read the contribution guidelines.
  • I have tested my changes locally.
  • I have added or updated unit tests where necessary.
  • I have updated the documentation if required.
  • My code follows the established coding standards.
  • I have added a news item linked to this PR.
  • I have reviewed CI checks for this PR and addressed them as far as I am able.

@seabbs seabbs enabled auto-merge (squash) November 19, 2024 12:16
@athowes athowes self-requested a review November 19, 2024 13:29
Copy link
Collaborator

@athowes athowes left a comment

Choose a reason for hiding this comment

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

Looks like this has everything apart from the :::

── R CMD check ─────────────────────────────────────────────────────────────────
* using log directory ‘/home/runner/work/epidist/epidist/check/epidist.Rcheck’
* using R version 4.4.2 (2024-10-31)
* using platform: x86_64-pc-linux-gnu
* R was compiled by
    gcc (Ubuntu 11.4.0-1ubuntu1~22.04) 11.4.0
    GNU Fortran (Ubuntu 11.4.0-1ubuntu1~22.04) 11.4.0
* running under: Ubuntu 22.04.5 LTS
* using session charset: UTF-8
* using options ‘--no-manual --ignore-vignettes --as-cran’
* checking for file ‘epidist/DESCRIPTION’ ... OK
* this is package ‘epidist’ version ‘0.0.0.9000’
* package encoding: UTF-8
* checking package namespace information ... OK
* checking package dependencies ... OK
* checking if this is a source package ... OK
* checking if there is a namespace ... OK
* checking for executable files ... OK
* checking for hidden files and directories ... OK
* checking for portable file names ... OK
* checking for sufficient/correct file permissions ... OK
* checking whether package ‘epidist’ can be installed ... OK
* checking installed package size ... NOTE
  installed size is 197.5Mb
  sub-directories of 1Mb or more:
    extdata  197.1Mb
* checking package directory ... OK
* checking for future file timestamps ... OK
* checking DESCRIPTION meta-information ... OK
* checking top-level files ... OK
* checking for left-over files ... OK
* checking index information ... OK
* checking package subdirectories ... OK
* checking code files for non-ASCII characters ... OK
* checking R files for syntax errors ... OK
* checking whether the package can be loaded ... OK
* checking whether the package can be loaded with stated dependencies ... OK
* checking whether the package can be unloaded cleanly ... OK
* checking whether the namespace can be loaded with stated dependencies ... OK
* checking whether the namespace can be unloaded cleanly ... OK
* checking loading without being on the library search path ... OK
* checking dependencies in R code ... NOTE
Unexported objects imported by ':::' calls:
  ‘brms:::dpar_bounds’ ‘brms:::validate_data’ ‘brms:::validate_family’
  ‘brms:::validate_formula’
  See the note in ?`:::` about the use of this operator.
* checking S3 generic/method consistency ... OK
* checking replacement functions ... OK
* checking foreign function calls ... OK
* checking R code for possible problems ... OK
* checking Rd files ... OK
* checking Rd metadata ... OK
* checking Rd line widths ... OK
* checking Rd cross-references ... OK
* checking for missing documentation entries ... OK
* checking for code/documentation mismatches ... OK
* checking Rd \usage sections ... OK
* checking Rd contents ... OK
* checking for unstated dependencies in examples ... OK
* checking contents of ‘data’ directory ... OK
* checking data for non-ASCII characters ... OK
* checking LazyData ... OK
* checking data for ASCII and uncompressed saves ... OK
* checking files in ‘vignettes’ ... SKIPPED
* checking examples ... NONE
* checking for unstated dependencies in ‘tests’ ... OK
* checking tests ...
  Running ‘testthat.R’
 OK
* checking for non-standard things in the check directory ... OK
* checking for detritus in the temp directory ... OK
* DONE

@seabbs seabbs merged commit 590838d into main Nov 19, 2024
9 checks passed
@seabbs seabbs deleted the fix-notes branch November 19, 2024 13:30
seabbs added a commit that referenced this pull request Jan 10, 2025
* Update .Rbuildignore

* Update .Rbuildignore

* Update DESCRIPTION

Former-commit-id: e58e62b79fa5a69c7a2aa09d3a23b189dcb0c062 [formerly b04de28ead6f218e03369ad61d98b00f0b50643d]
Former-commit-id: ad62ce7be529953d762ef5ad5f783a3fd45db5e6
seabbs added a commit that referenced this pull request Jan 21, 2025
* Update .Rbuildignore

* Update .Rbuildignore

* Update DESCRIPTION

Former-commit-id: 590838d
Former-commit-id: a2e5cde48f8999538b8387f58e6062dbc4e8f4c1
seabbs added a commit that referenced this pull request Jan 21, 2025
* Update .Rbuildignore

* Update .Rbuildignore

* Update DESCRIPTION

Former-commit-id: 590838d
Former-commit-id: a2e5cde48f8999538b8387f58e6062dbc4e8f4c1
seabbs added a commit that referenced this pull request Jan 21, 2025
* Update .Rbuildignore

* Update .Rbuildignore

* Update DESCRIPTION

Former-commit-id: 590838d
Former-commit-id: a2e5cde48f8999538b8387f58e6062dbc4e8f4c1
seabbs added a commit that referenced this pull request Jan 21, 2025
* Update .Rbuildignore

* Update .Rbuildignore

* Update DESCRIPTION

Former-commit-id: e58e62b79fa5a69c7a2aa09d3a23b189dcb0c062 [formerly b04de28ead6f218e03369ad61d98b00f0b50643d]
Former-commit-id: ad62ce7be529953d762ef5ad5f783a3fd45db5e6
seabbs added a commit that referenced this pull request Jan 21, 2025
* Update .Rbuildignore

* Update .Rbuildignore

* Update DESCRIPTION

Former-commit-id: e58e62b79fa5a69c7a2aa09d3a23b189dcb0c062 [formerly b04de28ead6f218e03369ad61d98b00f0b50643d]
Former-commit-id: ad62ce7be529953d762ef5ad5f783a3fd45db5e6
Former-commit-id: d4757f0
seabbs added a commit that referenced this pull request Jan 21, 2025
* Update .Rbuildignore

* Update .Rbuildignore

* Update DESCRIPTION

Former-commit-id: 590838d
Former-commit-id: a2e5cde48f8999538b8387f58e6062dbc4e8f4c1
Former-commit-id: 2298c3a1cda5420646e32ec29d22e1785ec4a035 [formerly 046ecae]
Former-commit-id: 3a9cb7ba65c3dce0ba2242b5e0d591aec68f4cc1
seabbs added a commit that referenced this pull request Jan 21, 2025
* Update .Rbuildignore

* Update .Rbuildignore

* Update DESCRIPTION

Former-commit-id: 590838d
Former-commit-id: a2e5cde48f8999538b8387f58e6062dbc4e8f4c1
Former-commit-id: 2298c3a1cda5420646e32ec29d22e1785ec4a035 [formerly 046ecae]
Former-commit-id: 3a9cb7ba65c3dce0ba2242b5e0d591aec68f4cc1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants