Skip to content

Conversation

@yutannihilation
Copy link
Contributor

@yutannihilation yutannihilation commented Dec 2, 2025

Close #394, close #393

While the update itself is almost no change in Rust code (so, this update is totally optional), the R wrapper is changed a bit. This pull request updates it manually by the following steps:

# If not installed yet.
install.packages(
  "savvy",
  repos = c("https://yutannihilation.r-universe.dev", "https://cloud.r-project.org")
)

# Download the latest savvy CLI
savvy::download_savvy_cli()

# Update the R wrapper files
savvy::savvy_update()

# Generate documents and NAMESPACE
devtools::document()

Copy link
Member

@paleolimbot paleolimbot left a comment

Choose a reason for hiding this comment

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

Thank you!

I'm not sure if there's a good way to embed the license file, but historically I've had to manually add it back in whenever I update the R rust code (perhaps you ran into that here too).

@paleolimbot paleolimbot merged commit 71c2786 into apache:main Dec 2, 2025
8 checks passed
@yutannihilation
Copy link
Contributor Author

Yeah, I did the same. Maybe I should make savvy support the license header, but manual edit is enough for now...

@yutannihilation yutannihilation deleted the chore/update-savvy-v0.9 branch December 2, 2025 22:38
pwrliang pushed a commit to pwrliang/sedona-db that referenced this pull request Dec 6, 2025
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