Skip to content

Upgrade dependencies#17

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

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

Conversation

@egtann

@egtann egtann commented Apr 22, 2026

Copy link
Copy Markdown
Contributor

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

coderabbitai Bot commented Apr 22, 2026

Copy link
Copy Markdown

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.

@mladeroute mladeroute 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.

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

egtann commented Apr 22, 2026

Copy link
Copy Markdown
Contributor Author

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

@mladeroute mladeroute 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.

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