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

test R package in CI/CD #25

Merged
merged 12 commits into from
Feb 6, 2024
Merged

test R package in CI/CD #25

merged 12 commits into from
Feb 6, 2024

Conversation

romainfrancois
Copy link
Collaborator

@romainfrancois romainfrancois commented Feb 6, 2024

closes #12

Copy link
Contributor

aviator-app bot commented Feb 6, 2024

Current Aviator status

Aviator will automatically update this comment as the status of the PR changes.
Comment /aviator refresh to force Aviator to re-examine your PR (or learn about other /aviator commands).

This PR was merged manually (without Aviator). Merging manually can negatively impact the performance of the queue. Consider using Aviator next time.


See the real-time status of this PR on the Aviator webapp.
Use the Aviator Chrome Extension to see the status of your PR within GitHub.

@romainfrancois
Copy link
Collaborator Author

romainfrancois commented Feb 6, 2024

This seems to fail because of a mismatch of version of duckdb. https://github.com/krlmlr/duckdb-rfuns/actions/runs/7798916569/job/21268579804?pr=25#step:6:156

Maybe something is wrong is how this uses the actions/checkout

I'll get inspired from what is done here: https://github.com/duckdb/duckdb/blob/4f040413468b2fee84ce4d124dfb058378db1456/.github/workflows/_extension_client_tests.yml#L29

@romainfrancois romainfrancois marked this pull request as ready for review February 6, 2024 13:41
@romainfrancois romainfrancois changed the title [wip] test R package in actions test R package in CI/CD Feb 6, 2024
@aviator-app aviator-app bot added the blocked label Feb 6, 2024
Copy link
Contributor

aviator-app bot commented Feb 6, 2024

This pull request failed to merge: PR cannot be automatically rebased, please rebase manually to continue. Remove the blocked label to re-queue.

Additional debug info: Failed to rebase this PR onto the latest changes from the base branch. You will probably need to rebase this PR manually and resolve conflicts).

Copy link
Collaborator

@krlmlr krlmlr left a comment

Choose a reason for hiding this comment

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

Thanks!

@krlmlr krlmlr removed the blocked label Feb 6, 2024
@aviator-app aviator-app bot added the blocked label Feb 6, 2024
Copy link
Contributor

aviator-app bot commented Feb 6, 2024

This pull request failed to merge: PR cannot be automatically rebased, please rebase manually to continue. Remove the blocked label to re-queue.

Additional debug info: Failed to rebase this PR onto the latest changes from the base branch. You will probably need to rebase this PR manually and resolve conflicts).

@krlmlr krlmlr merged commit 23ddc30 into main Feb 6, 2024
22 of 23 checks passed
@krlmlr krlmlr deleted the check_pkg branch February 6, 2024 17:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Test R package in CI/CD
2 participants