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

ARROW-11700: [R] Internationalize error handling in tidy eval #9717

Closed
wants to merge 2 commits into from

Conversation

nealrichardson
Copy link
Member

Also fixes ARROW-11785 along the way.

@github-actions
Copy link

.data$filtered_rows <- new_filter
} else {
.data$filtered_rows <- .data$filtered_rows & new_filter
if (length(expressions)) {
Copy link
Member Author

Choose a reason for hiding this comment

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

FYI @ianmcook this is the fix for ARROW-11785

r/R/dplyr.R Outdated Show resolved Hide resolved
Copy link
Contributor

@pachadotdev pachadotdev left a comment

Choose a reason for hiding this comment

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

Hi. This makes perfect sense in my opinion. My computer is in "en_US:en", but I'm used to á é í ó ú ñ, and how i18n is used here provides a standard that looks very stable.

@nealrichardson nealrichardson deleted the i18n branch March 22, 2021 18:15
GeorgeAp pushed a commit to sirensolutions/arrow that referenced this pull request Jun 7, 2021
Also fixes ARROW-11785 along the way.

Closes apache#9717 from nealrichardson/i18n

Authored-by: Neal Richardson <neal.p.richardson@gmail.com>
Signed-off-by: Neal Richardson <neal.p.richardson@gmail.com>
michalursa pushed a commit to michalursa/arrow that referenced this pull request Jun 13, 2021
Also fixes ARROW-11785 along the way.

Closes apache#9717 from nealrichardson/i18n

Authored-by: Neal Richardson <neal.p.richardson@gmail.com>
Signed-off-by: Neal Richardson <neal.p.richardson@gmail.com>
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.

None yet

2 participants