Skip to content

Go 1.26.1#98

Open
az-smartling wants to merge 3 commits intomasterfrom
go_1_26_1
Open

Go 1.26.1#98
az-smartling wants to merge 3 commits intomasterfrom
go_1_26_1

Conversation

@az-smartling
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the project to Go 1.26.1, replaces interface{} with any, and adopts modern Go features like t.Context() and strings.SplitSeq. It also updates YAML tags to use omitzero. However, a critical syntax error was introduced in the machine translation services where pointer.NewP(value) was incorrectly replaced with new(value), which will cause compilation failures because the new built-in function requires a type argument rather than a value.

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.

1 participant