Skip to content
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

chore: fix function name #3562

Closed
wants to merge 2 commits into from
Closed

chore: fix function name #3562

wants to merge 2 commits into from

Conversation

finaltrip
Copy link

Overview

fix function name

What this PR does / why we need it

Special notes for your reviewer

Signed-off-by: finaltrip <[email protected]>
@nabokihms nabokihms added the release-note/bug-fix Release note: Bug Fixes label Jun 3, 2024
@nabokihms
Copy link
Member

This code is autogenerated I'm afraid.

@cpanato
Copy link
Contributor

cpanato commented Jun 4, 2024

that is wrong in the template from https://github.com/ent/ent (https://github.com/ent/ent/blob/092b62a3229476d18d9ffade87da268d9136f13a/entc/gen/template/dialect/sql/by.tmpl#L19) we should fix there

@cpanato
Copy link
Contributor

cpanato commented Jun 4, 2024

PR that fix in upstream: ent/ent#4083

@finaltrip
Copy link
Author

PR that fix in upstream: ent/ent#4083

Thanks!

@cpanato
Copy link
Contributor

cpanato commented Jul 19, 2024

fixed, maybe we can rebase?

@finaltrip
Copy link
Author

fixed, maybe we can rebase?

Updated. Please review again.

@nabokihms
Copy link
Member

There are no changes in this PR anymore. Do we still need these two commits to be merged?

@cpanato
Copy link
Contributor

cpanato commented Oct 18, 2024

yep we can close it

@nabokihms
Copy link
Member

Thanks @cpanato

@nabokihms nabokihms closed this Oct 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/bug-fix Release note: Bug Fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants