Skip to content

refactor: introduce alias for component cblock modeloutputthunk union - #1458

Open
AngeloDanducci wants to merge 1 commit into
generative-computing:mainfrom
AngeloDanducci:ad-1439
Open

refactor: introduce alias for component cblock modeloutputthunk union#1458
AngeloDanducci wants to merge 1 commit into
generative-computing:mainfrom
AngeloDanducci:ad-1439

Conversation

@AngeloDanducci

Copy link
Copy Markdown
Contributor

Pull Request

Issue

Fixes #1439

Description

Introduces a new alias (NodeData) for the Component | Cblock | ModelOutputThunk union.

Also removes old imports no longer necessary based on this refactor.

New name open to discussion.

Testing

  • Tests added to the respective file if code was changed
  • New code has 100% coverage if code was added
  • Ensure existing tests and github automation passes (a maintainer will kick off the github automation when the rest of the PR is populated)

Attribution

  • AI coding assistants used

Adding a new component, requirement, sampling strategy, or tool?

If your PR adds or modifies one of the types below, check the matching box. A checklist of type-specific review items will be posted as a comment.

  • Component
  • Requirement
  • Sampling Strategy
  • Tool

NOTE: Please ensure you have an issue that has been acknowledged by a core contributor and routed you to open a pull request against this repository. Otherwise, please open an issue before continuing with this pull request.

Signed-off-by: AngeloDanducci <angelo.danducci.ii@ibm.com>
@github-actions github-actions Bot added the enhancement New feature or request label Jul 28, 2026
@AngeloDanducci
AngeloDanducci marked this pull request as ready for review July 29, 2026 14:49
@AngeloDanducci
AngeloDanducci requested a review from a team as a code owner July 29, 2026 14:49

@akihikokuroda akihikokuroda 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.

LGTM

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

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

refactor: introduce shared alias for the Component | CBlock | ModelOutputThunk union

2 participants