Skip to content

chore(security): Upgrade Go toolchain and deps [DEVOP-8052] - #47

Open
ChristianWitts wants to merge 1 commit into
mainfrom
christian/devop-8052-go-upgrade-security
Open

chore(security): Upgrade Go toolchain and deps [DEVOP-8052]#47
ChristianWitts wants to merge 1 commit into
mainfrom
christian/devop-8052-go-upgrade-security

Conversation

@ChristianWitts

Copy link
Copy Markdown

Description

  • Go: root 1.26→1.26.0, examples/graphql_flow 1.17→1.26.0
  • Deps: go.mod/go.sum updated
  • Actions: pinned (build.yaml, semantic-pr.yaml)
  • Known manual fix needed: examples/graphql_flow — gqlgen API breaking change (IChallenge type and Dispatch signature changed); example code only, does not affect the library itself

Refs: #DEVOP-8052

Signed-off-by: Christian Witts christian@honestbank.com

  - Go: root 1.26→1.26.0, examples/graphql_flow 1.17→1.26.0
  - Deps: go.mod/go.sum updated
  - Actions: pinned (build.yaml, semantic-pr.yaml)
  - Known manual fix needed: examples/graphql_flow — gqlgen API breaking change (IChallenge type and Dispatch signature changed); example code only, does not affect the library itself

Refs: #DEVOP-8052

Signed-off-by: Christian Witts <christian@honestbank.com>
@ChristianWitts
ChristianWitts requested review from a team as code owners May 25, 2026 10:49
@ChristianWitts
ChristianWitts enabled auto-merge (squash) May 25, 2026 11:13

@applerogue applerogue left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Review

Rechecked head 06c811ed1836.

  • Go toolchain/deps bump + pinned Actions (DEVOP-8052) look reasonable; go-tests / CodeQL / semantic-release are green.
  • Author note about examples/graphql_flow gqlgen API break is clear (example-only).
  • CI blocker: Mergeable failed (only failing check).

Verdict

Request changes — please resolve Mergeable on this head (checklist / policy), then re-request review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants