Skip to content

Restrict visibility of some extra class declarations#214

Merged
tim-hoffman merged 2 commits intomainfrom
th/reduce_visibility
Nov 14, 2025
Merged

Restrict visibility of some extra class declarations#214
tim-hoffman merged 2 commits intomainfrom
th/reduce_visibility

Conversation

@tim-hoffman
Copy link
Member

The functions for custom printing and parsing rules do not need to be public, they are only used within the current op implementation.

@tim-hoffman tim-hoffman requested a review from a team November 13, 2025 21:11
Copy link
Contributor

@iangneal iangneal left a comment

Choose a reason for hiding this comment

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

LGTM

@tim-hoffman tim-hoffman merged commit 5244f87 into main Nov 14, 2025
10 checks passed
@tim-hoffman tim-hoffman deleted the th/reduce_visibility branch November 14, 2025 17:44
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