New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
transformations: Add a flag for target selection on stencil conversion. #1031
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I pressed the update branch
button, that seems to have merged main into this branch. I hope I did not break anything, @PapyChacal. Might be a dumb idea to do that as a non-author, so sorry 😅
Other than that, LGTM.
Okay, I definitely broke something, whups! |
Can you force push your local branch before pulling? Or do the merge locally and then force push... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks fine, I guess you need to fix/update first
Thanks for offering to take the blame, but I did something wrong before you 😛 |
d7ae3fe
to
0d4521d
Compare
Get rid of
convert-stencil-to-gpu
as it can be a simple flag now.Use a
Literal
flag type to enforce specific values only.Add tests for that case.
Stacked on #1030