Skip to content

chore: add sub tags - #10473

Merged
Jefffrey merged 5 commits into
apache:mainfrom
Rich-T-kid:rich-T-kid/add-crate-tags
Jul 30, 2026
Merged

chore: add sub tags#10473
Jefffrey merged 5 commits into
apache:mainfrom
Rich-T-kid:rich-T-kid/add-crate-tags

Conversation

@Rich-T-kid

Copy link
Copy Markdown
Contributor

Which issue does this PR close?

Rationale for this change

see #10435 (comment)

What changes are included in this PR?

adds sub tags for each crate touched in a PR

Are these changes tested?

This PR serves as a test

Are there any user-facing changes?

yes, contributers will have an easier time finding issues/PR's in crates they are interested in through github search

Comment thread .github/workflows/dev_pr/labeler.yml Outdated
Comment on lines +20 to +22
- any-glob-to-any-file:
- "arrow-arith/**/*"
- "arrow-array/**/*"

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I can remove the extra space.

Im not sure why `` wasnt working on my macbook. the file wouldnt stop erroring unless I used ""

@alamb alamb added the development-process Related to development process of arrow-rs label Jul 29, 2026
@Jefffrey

Copy link
Copy Markdown
Contributor

the indentation changes look a bit off

is your editor defaulting to 4 spaces?

@Rich-T-kid
Rich-T-kid force-pushed the rich-T-kid/add-crate-tags branch from 10f1a53 to 83d19d0 Compare July 29, 2026 13:40
@Rich-T-kid

Copy link
Copy Markdown
Contributor Author

it should be fixed now.

@Jefffrey Jefffrey left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

ive created some of these labels now, though i feel we can group some of them together as we dont need to be too granular

Comment thread .github/workflows/dev_pr/labeler.yml Outdated
- any-glob-to-any-file:
- "arrow-integration-test/**/*"

arrow-integration-testing:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

can probably group this in with arrow-integration-test above

Comment thread .github/workflows/dev_pr/labeler.yml
Comment thread .github/workflows/dev_pr/labeler.yml Outdated
@Rich-T-kid
Rich-T-kid force-pushed the rich-T-kid/add-crate-tags branch from f3d1eeb to b8e5cde Compare July 30, 2026 15:06
@Rich-T-kid

Copy link
Copy Markdown
Contributor Author

@Jefffrey I trimmed the PR a bit

@Jefffrey
Jefffrey merged commit 23e61b6 into apache:main Jul 30, 2026
38 checks passed
@Jefffrey

Copy link
Copy Markdown
Contributor

thanks @Rich-T-kid

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

development-process Related to development process of arrow-rs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Can we add crate tags for issues/PR's

3 participants