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.

Sign up for GitHub

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

Jump to bottom

Rename R to R-arrow #68

Merged
merged 4 commits into from
Dec 6, 2023
Merged

Rename R to R-arrow #68

merged 4 commits into from
Dec 6, 2023
+42 −35

Conversation

Tmonster
Copy link
Collaborator

@Tmonster Tmonster commented Dec 4, 2023

No description provided.

Tmonster and others added 4 commits November 22, 2023 13:32
@Tmonster
should change arrow to show R-arrow
a493b3b
@Tmonster
new arrow benchmarks report solution as R-arrow
cf48b23
@Tmonster
update arrow to R-arrow in a few more places
5227ea5
@amoeba
Fix remaining issues in #10 (#13)
3d136f8 
* Fix remaining issues in arrow -> R-arrow rename

* Fix bug in rename code in report.R

The previous code was causing something wild to happen. The changed code is idiomatic code for replacing values in a data.frame based on a condition.
@Tmonster Tmonster mentioned this pull request Dec 4, 2023
Closed
@Tmonster Tmonster merged commit c84afb2 into duckdblabs:master Dec 6, 2023
14 checks passed
@Tmonster
Copy link
Collaborator Author

Tmonster commented Dec 6, 2023

Results with R-arrow should be up!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
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
@Tmonster @amoeba