Skip to content

Update R-CMD-check.yaml #17

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.

Sign up for GitHub

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

Jump to bottom
Merged
francisbarton merged 29 commits into from
Aug 19, 2025
Merged

Update R-CMD-check.yaml #17

francisbarton merged 29 commits into from
Aug 19, 2025

Conversation

@francisbarton
Copy link
Collaborator

@francisbarton francisbarton commented Aug 15, 2025

Remove trigger on push to main (as we don't do that!) Try adding in httpuv dependency to see if that helps CMD check pass

@francisbarton
Update R-CMD-check.yaml
82a1329 
Remove trigger on push to main (as we don't do that!)
Try adding in httpuv dependency to see if that helps CMD check pass
@francisbarton francisbarton linked an issue Aug 15, 2025 that may be closed by this pull request
Fix RCMD GHA yaml so it succeeds! #16
Closed
@francisbarton
🔀 Switch from {jsonlite} to {yyjsonr}
74578de
@francisbarton
📝 Update Description text
8849051
@francisbarton
⬆️ Set min R version to 4.4
a3af0c7
@francisbarton
✅ Make test with {withr} conditional
fa95b61
@francisbarton
✅ Basic functionality testing for read_* functions
d36d929
@francisbarton
🤕 Bump version
b381d58
@francisbarton
Refactor envvar checking in get_container() etc
cda66fe
@francisbarton
Refactor envvar checking in get_container() etc
af8bddb 
Fixes #22
@francisbarton
Add tests for behaviour of check_scalar_type()
7a6842b
@francisbarton
➕ Add gregg() helper
327bdd6
@francisbarton
⚙ Add list_files function
b1c3b5d 
Should close #26
@francisbarton
🧹 Tidy up code in a couple of tests
dc8dda8
@francisbarton
Add usage instructions to README
f92e31a 
Closes #18
@francisbarton
🧹 Tidy up code in a couple of tests
bc1b0dc 
🧹  Tidy up
@francisbarton
🔧Fix naming of stopifnot checks
df2b978
@francisbarton
🔧 Actually fix naming of stopifnot checks
80a2163
@francisbarton
📝 Small changes to function docn.
24addf9
@francisbarton
Add tests for reading RDS files
c2596a1
@francisbarton
Split out check_blob_exists function
20013e3
@francisbarton
Use check_blob_exists to return filename instead of raw data for csv …
6f605e3 
…and rds
@francisbarton
Rearrange order of arguments
3032f95
@francisbarton
🔃 rename helper function
d3f559d
@francisbarton
✅ Add overall basic tests for read_* functions
dd8bd2e
@francisbarton
🗺️ Update README
cfce91e
@francisbarton
🖌️ Improvements to function docs and examples
de6f0da
@francisbarton
Addresses failing read_azure_rds() and missing tests. Bump to v0.1
7b0271a 
Should close #32
@francisbarton
Add skip_on_ci()
1de9a31
@francisbarton
Merge branch 'main' into 16-change-r-cmd-check-gha-yaml
8208232
@francisbarton
Copy link
Collaborator Author

francisbarton commented Aug 19, 2025

testthat::skip_on_ci() added to a test helped this pass (thanks Rhian!)

@francisbarton francisbarton marked this pull request as ready for review August 19, 2025 16:59
@francisbarton francisbarton merged commit b2dd338 into main Aug 19, 2025
1 check passed
@francisbarton francisbarton deleted the 16-change-r-cmd-check-gha-yaml branch August 19, 2025 22:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tomjemmett tomjemmett tomjemmett approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Fix RCMD GHA yaml so it succeeds!

3 participants

@francisbarton @tomjemmett