Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

[r] Remove two more tiledb::schema callsites #3160

Merged
merged 2 commits into from
Oct 10, 2024
Merged

[r] Remove two more tiledb::schema callsites #3160

merged 2 commits into from
Oct 10, 2024

Conversation

johnkerl
Copy link
Member

@johnkerl johnkerl commented Oct 10, 2024 •
edited
Loading

@johnkerl johnkerl changed the title [r] Remove two more tiledb::schema callsites [r] Remove three more tiledb::schema callsites Oct 10, 2024
@johnkerl johnkerl changed the title [r] Remove three more tiledb::schema callsites [r] Remove two more tiledb::schema callsites Oct 10, 2024
mojaveazure
mojaveazure approved these changes Oct 10, 2024
View reviewed changes
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; please update the changelog, bump the develop version, and 🚢

@johnkerl johnkerl merged commit cced6ea into main Oct 10, 2024
@johnkerl johnkerl deleted the kerl/more-tdr-schema branch October 10, 2024 14:47
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@mojaveazure mojaveazure mojaveazure 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.
2 participants
@johnkerl @mojaveazure