Skip to content

Java: Fix sink model generator for instance parameters#12969

Merged
atorralba merged 1 commit intogithub:mainfrom
atorralba:atorralba/java/fix-model-generator-sinks-instance-parameters
May 2, 2023
Merged

Java: Fix sink model generator for instance parameters#12969
atorralba merged 1 commit intogithub:mainfrom
atorralba:atorralba/java/fix-model-generator-sinks-instance-parameters

Conversation

@atorralba
Copy link
Copy Markdown
Contributor

This PR fixes the model generator when it's generating sinks that are InstanceParameterNodes. Previously it generated input columns like Argument[-1], but they should be Argument[this].

@atorralba atorralba requested a review from a team as a code owner April 28, 2023 12:51
@atorralba atorralba added the no-change-note-required This PR does not need a change note label Apr 28, 2023
@github-actions github-actions bot added the Java label Apr 28, 2023
Copy link
Copy Markdown
Contributor

@michaelnebel michaelnebel left a comment

Choose a reason for hiding this comment

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

LGTM!

@atorralba atorralba merged commit 51c08f1 into github:main May 2, 2023
@atorralba atorralba deleted the atorralba/java/fix-model-generator-sinks-instance-parameters branch May 2, 2023 08:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Java no-change-note-required This PR does not need a change note

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants