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

Update vignettes/job_logging.Rmd to fix build errors #142

Merged
merged 1 commit into from
Nov 17, 2023

Conversation

pbchase
Copy link
Contributor

@pbchase pbchase commented Nov 17, 2023

Address dbGetInfo errors in the iris-updater chunk. specify duckDB when initting.

Addresses Isue #141

Address dbGetInfo errors in the iris-updater chunk.
specify duckDB when initting.
@pbchase pbchase merged commit b5cbe53 into ctsit:develop Nov 17, 2023
1 check passed
@ChemiKyle
Copy link
Contributor

Merging your own PR, how uncouth!

@pbchase
Copy link
Contributor Author

pbchase commented Nov 19, 2023

Merging your own PR, how uncouth!

I agree. I thought the checks run in the PR were equivalent to those run when merged to the main/master, but they aren't. When the PR failed to run the tests I needed, I merged it and released it. Sadly, I got this link in my inbox: https://github.com/ctsit/redcapcustodian/actions/runs/6909526435

I feel like I need to stop evaluating the code in those chunks and stop referring to the data objects that will not be generated as a result. :-(

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