Skip to content

chore: remove obsolete allow(dead_code) annotations#206

Merged
JingsongLi merged 1 commit intoapache:mainfrom
luoyuxia:remove-dead-code
Apr 5, 2026
Merged

chore: remove obsolete allow(dead_code) annotations#206
JingsongLi merged 1 commit intoapache:mainfrom
luoyuxia:remove-dead-code

Conversation

@luoyuxia
Copy link
Copy Markdown
Contributor

@luoyuxia luoyuxia commented Apr 4, 2026

Purpose

Linked issue: close #xxx

This PR removes allow(dead_code) annotations that were introduced earlier during implementation to temporarily suppress warnings for code that had not yet been wired up.

As the related code paths are now being used, these annotations are no longer necessary. Cleaning them up makes the codebase tidier and helps keep warning suppression limited to cases where it is actually needed.

Brief change log

Tests

API and Format

Documentation

Copy link
Copy Markdown
Contributor

@JingsongLi JingsongLi left a comment

Choose a reason for hiding this comment

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

+1

@JingsongLi JingsongLi merged commit 9e59782 into apache:main Apr 5, 2026
8 checks passed
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