Skip to content

Upgrade dependencies#17

Merged
egtann merged 1 commit intomasterfrom
evantann/sc-235717/sagansystems-migrate-github-com-thankful-ai
Apr 22, 2026
Merged

Upgrade dependencies#17
egtann merged 1 commit intomasterfrom
evantann/sc-235717/sagansystems-migrate-github-com-thankful-ai

Conversation

@egtann
Copy link
Copy Markdown
Contributor

@egtann egtann commented Apr 22, 2026

why

Fix Snyk-flagged vulnerabilities and bring the remaining dependencies up-to-date at the same time.

testing

NMTN.

release notes

N/A

@egtann egtann self-assigned this Apr 22, 2026
@saganbot
Copy link
Copy Markdown

Storyhttps://app.shortcut.com/gladly/story/235717
Review Guidehttps://www.notion.so/gladly/Change-Approvals-at-Gladly-37814c3e0da8411985f6a1adaa2f64ad
RollbackFor repos on weekly release: roll back the release. For repos on continuous deployment (CD): revert the code change and deploy.

@saganbot saganbot added the chore label Apr 22, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 22, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 3783e361-7882-4462-aaa7-8259a35cb5df

📥 Commits

Reviewing files that changed from the base of the PR and between 34a48e3 and 21fc128.

⛔ Files ignored due to path filters (2)
  • go.mod is excluded by !**/*.mod
  • go.sum is excluded by !**/*.sum, !**/*.sum
📒 Files selected for processing (1)
  • Dockerfile

📝 Walkthrough

Walkthrough

The Dockerfile's build stage base image is upgraded from golang:1.22-alpine to golang:1.26-alpine, while maintaining the --platform=$BUILDPLATFORM flag. All subsequent build instructions—including clang/lld installation, CGO configuration, module downloads, static binary compilation, and verification—remain identical.


Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@mladeroute mladeroute left a comment

Choose a reason for hiding this comment

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

Build error for image:

Error: buildx failed with: ERROR: failed to build: failed to solve: process "/bin/sh -c xx-go mod download -x" did not complete successfully: exit code: 1```

@egtann egtann force-pushed the evantann/sc-235717/sagansystems-migrate-github-com-thankful-ai branch from 3007326 to 21fc128 Compare April 22, 2026 16:34
@egtann
Copy link
Copy Markdown
Contributor Author

egtann commented Apr 22, 2026

Upgraded Go in the Dockerfile to exceed the version required in go.mod, should fix the issue in CI.

Copy link
Copy Markdown

@mladeroute mladeroute left a comment

Choose a reason for hiding this comment

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

LGTM!

@egtann egtann merged commit f653ef7 into master Apr 22, 2026
7 checks passed
@egtann egtann deleted the evantann/sc-235717/sagansystems-migrate-github-com-thankful-ai branch April 22, 2026 16:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

3 participants