Skip to content

Conversation

@HyukjinKwon
Copy link
Member

Rationale for this change

There are a TODO for documentation which result in https://arrow.apache.org/docs/r/reference/DictionaryType.html

What changes are included in this PR?

This PR documents DictionaryType classes.

Are these changes tested?

I did not test them as they are documentation changes.

Are there any user-facing changes?

Yes, it fixes the user facing documentation.

#' @include type.R

#' @title class DictionaryType
#' @title DictionaryType class
Copy link
Member

Choose a reason for hiding this comment

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

Good catch on this typo

Copy link
Member

@jonkeane jonkeane left a comment

Choose a reason for hiding this comment

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

Thanks

@github-actions github-actions bot added awaiting changes Awaiting changes and removed awaiting review Awaiting review labels Dec 27, 2025
@jonkeane jonkeane merged commit c4c4c0b into apache:main Dec 27, 2025
11 checks passed
@github-actions github-actions bot added awaiting merge Awaiting merge and removed awaiting changes Awaiting changes labels Dec 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants