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

fix/port 590 #630

Merged
merged 4 commits into from
Jan 2, 2025
Merged

fix/port 590 #630

merged 4 commits into from
Jan 2, 2025

Conversation

baywet
Copy link
Member

@baywet baywet commented Dec 24, 2024

port of #590 and #595 to the v1 branch

irvinesunday and others added 3 commits December 24, 2024 13:04
…ctions (#590)

* Update unit function operation test

* Account for overloaded functions in any segment

* Update release note

Signed-off-by: Vincent Biret <[email protected]>
…overloaded functions where all functions in path are overloaded (#595)

* Ensure unique operation ids for composable functions with both overloaded

* Update test to validate fix

* Update release notes

Signed-off-by: Vincent Biret <[email protected]>
darrelmiller
darrelmiller previously approved these changes Dec 24, 2024
andrueastman
andrueastman previously approved these changes Jan 2, 2025
@baywet baywet dismissed stale reviews from andrueastman and darrelmiller via 73ff328 January 2, 2025 13:22
@baywet baywet enabled auto-merge January 2, 2025 13:23
@baywet baywet requested a review from andrueastman January 2, 2025 13:23
@baywet baywet merged commit 99e6a4a into support/v1 Jan 2, 2025
14 checks passed
@baywet baywet deleted the fix/port-590 branch January 2, 2025 13:38
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.

4 participants