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

Add wrapper for vfs_copy_dir() function #804

Merged
merged 1 commit into from
Feb 13, 2025

Conversation

nickvigilante
Copy link
Contributor

I noticed this function was missing when writing up a tutorial on VFS, so this is my attempt at wrapping it.

@nickvigilante nickvigilante force-pushed the nv/add-tiledb_vfs_copy_dir branch from 0379527 to 56c62f1 Compare February 13, 2025 14:20
R/VFS.R Show resolved Hide resolved
R/VFS.R Outdated Show resolved Hide resolved
@nickvigilante nickvigilante force-pushed the nv/add-tiledb_vfs_copy_dir branch from 56c62f1 to 9fa5746 Compare February 13, 2025 16:05
@johnkerl johnkerl changed the title Added wrapper for vfs_copy_dir() function Add wrapper for vfs_copy_dir() function Feb 13, 2025
@nickvigilante nickvigilante force-pushed the nv/add-tiledb_vfs_copy_dir branch 2 times, most recently from 4ab2f8f to 5cbca18 Compare February 13, 2025 16:29
@nickvigilante nickvigilante marked this pull request as ready for review February 13, 2025 16:29
@nickvigilante nickvigilante force-pushed the nv/add-tiledb_vfs_copy_dir branch 2 times, most recently from 63ea055 to 67af183 Compare February 13, 2025 16:52
Copy link
Member

@mojaveazure mojaveazure left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for adding! Please bump the develop version in DESCRIPTION (as of writing, should bump to 0.31.0.2) and add a changelog item to NEWS.md before merging

Copy link
Collaborator

@johnkerl johnkerl left a comment

Choose a reason for hiding this comment

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

🚀

Nice work @nickvigilante !!!!! :)

I noticed this function was missing when writing up a tutorial on
VFS, so this is my attempt at wrapping it.
@nickvigilante nickvigilante force-pushed the nv/add-tiledb_vfs_copy_dir branch from 67af183 to b99a559 Compare February 13, 2025 19:07
@nickvigilante nickvigilante merged commit b08b4c6 into main Feb 13, 2025
13 checks passed
@nickvigilante nickvigilante deleted the nv/add-tiledb_vfs_copy_dir branch February 13, 2025 20:37
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.

4 participants