Conversation
brownag
commented
Oct 18, 2025
- PR for patch release that will close [bug] file asset access issues on CRAN #135
There was a problem hiding this comment.
Pull Request Overview
This PR fixes an issue where sample DEM data was being excluded from the R package build by modifying the .Rbuildignore file to allow specific test data files while still excluding other TIFF files.
- Updated .Rbuildignore pattern to exclude TIFF files except for the specific DEM test data file
- Added defensive test code to handle cases where sample data files might not exist
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| .Rbuildignore | Modified regex pattern to allow inst/extdata/DEM.tif while excluding other TIFF files |
| tests/testthat/test-wbt_source.R | Added error handling and file existence checks for sample data |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.