Skip to content

Conversation

@tommysitu
Copy link
Member

No description provided.

@stuioco stuioco changed the title Templating header manipulation Templating functions - setHeader, initArray, variadic concat Aug 27, 2025
} else {
// @todo Maybe we can panic on that
return reflect.Zero(strType)
if funcType.IsVariadic() {
Copy link
Member Author

Choose a reason for hiding this comment

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

have we manually updated this file in vendor package?

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes, in order to support variadic template helpers

Copy link
Member Author

Choose a reason for hiding this comment

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

Do you get this code from the official Raymond library? If this is our own implementation, we need to update the fork: https://github.com/SpectoLabs/raymond and then bump the version in hoverfly go.mod.

Because vendor package is maintened by running go mod vendor and it will override any manual change.

@tommysitu tommysitu requested a review from stuioco September 5, 2025 08:04
@tommysitu tommysitu force-pushed the templating-header-manipulation branch from ed80a9f to f69e5ce Compare September 10, 2025 18:58
@tommysitu tommysitu merged commit be277f7 into master Sep 10, 2025
2 of 3 checks passed
@tommysitu tommysitu deleted the templating-header-manipulation branch September 10, 2025 19:01
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.

3 participants