Skip to content

feat: update package#44

Merged
AndrewHanasiro merged 2 commits intomainfrom
feature/update-package
Apr 15, 2026
Merged

feat: update package#44
AndrewHanasiro merged 2 commits intomainfrom
feature/update-package

Conversation

@AndrewHanasiro
Copy link
Copy Markdown
Member

@AndrewHanasiro AndrewHanasiro commented Apr 15, 2026

Summary by CodeRabbit

  • Chores
    • Updated Go runtime from version 1.24 to 1.26 in build pipeline
    • Upgraded multiple project dependencies to latest compatible versions
    • Modified infrastructure cleanup process to remove associated volumes during teardown

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 15, 2026

Warning

Rate limit exceeded

@AndrewHanasiro has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 55 minutes and 3 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 55 minutes and 3 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 30554dd3-257a-490e-89f9-d76dda6f8b2b

📥 Commits

Reviewing files that changed from the base of the PR and between b4c52d9 and a958212.

📒 Files selected for processing (2)
  • .github/workflows/continuous_integration.yml
  • .github/workflows/manual_deploy.yml
📝 Walkthrough

Walkthrough

Updated Go toolchain from version 1.24 to 1.26 in the Dockerfile build stage and go.mod configuration. Bumped multiple dependencies to newer versions, including Firebase SDK, Gin, Kafka-go, Twilio, and OpenTelemetry packages. Modified the infra/down target to remove named volumes during teardown.

Changes

Cohort / File(s) Summary
Go Toolchain Upgrade
Dockerfile
Updated Go builder base image from golang:1.24-alpine to golang:1.26-alpine for build stages.
Dependency Updates
go.mod
Upgraded Go toolchain from 1.24.0 to 1.26.0 and refreshed multiple direct and indirect dependencies (Firebase SDK, Gin, Kafka-go, Twilio, OpenTelemetry, Google Cloud libraries, etc.). Introduced go.mongodb.org/mongo-driver/v2 as new indirect dependency.
Infrastructure Configuration
makefile
Modified infra/down target to use docker compose down -v for removing associated named volumes during teardown.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~15 minutes

Possibly related PRs

  • feat: update #42: Touches the same files (Dockerfile and go.mod) to upgrade Go toolchain and refresh dependency versions.

Poem

🐰 Hop to 1.26, the Go version leaps so high!
Dependencies dance with each update nearby,
Volumes vanish with -v's gentle touch,
This rabbit's whiskers twitch — it's building so much! ✨

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name Status Explanation Resolution
Description check ⚠️ Warning No pull request description was provided by the author, leaving the required proposal and links sections completely empty. Add a complete description following the template with proposal details explaining the Go/dependency upgrades, rationale, and any relevant links or issue references.
Title check ❓ Inconclusive The title 'feat: update package' is vague and generic, failing to specify what package was updated or the nature of the update (Go version, dependencies, build configuration). Revise the title to be more specific, e.g., 'feat: upgrade Go toolchain to 1.26 and update dependencies' to clearly convey the main change.
✅ Passed checks (1 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/update-package

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🧹 Nitpick comments (1)
makefile (1)

7-7: Consider separating destructive teardown from default infra/down.

Line 7 now always removes named volumes (-v), which can unexpectedly wipe local state. Prefer keeping infra/down non-destructive and adding a dedicated target like infra/down/full for volume removal.

Suggested Makefile adjustment
 .PHONY: infra/down
 infra/down:
-	docker compose down -v
+	docker compose down
+
+.PHONY: infra/down/full
+infra/down/full:
+	docker compose down -v
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@makefile` at line 7, Update the Makefile so the default infra/down target is
non-destructive by removing the volume- removal flag from the existing docker
compose down command (replace the current `docker compose down -v` usage in the
`infra/down` target with `docker compose down`), and add a new target named
`infra/down/full` that runs `docker compose down -v` to explicitly remove named
volumes; ensure target names `infra/down` and `infra/down/full` are used exactly
so callers can opt-in to the destructive teardown.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@go.mod`:
- Line 3: CI is failing because go.mod requires Go 1.26.0 while the workflows
still pin go-version: "1.24.0"; update the go-version keys in
.github/workflows/continuous_integration.yml (the two occurrences currently set
via the go-version entries) to use "1.26.0" or a flexible constraint like
"^1.26" so the workflow matches go.mod and CI will run with the correct Go
toolchain.

---

Nitpick comments:
In `@makefile`:
- Line 7: Update the Makefile so the default infra/down target is
non-destructive by removing the volume- removal flag from the existing docker
compose down command (replace the current `docker compose down -v` usage in the
`infra/down` target with `docker compose down`), and add a new target named
`infra/down/full` that runs `docker compose down -v` to explicitly remove named
volumes; ensure target names `infra/down` and `infra/down/full` are used exactly
so callers can opt-in to the destructive teardown.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 7088f363-898c-4bb3-b307-3a2d84dac455

📥 Commits

Reviewing files that changed from the base of the PR and between 2fdd4a4 and b4c52d9.

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

Comment thread go.mod
@sonarqubecloud
Copy link
Copy Markdown

@AndrewHanasiro AndrewHanasiro merged commit 0ed9b3d into main Apr 15, 2026
12 of 15 checks passed
@AndrewHanasiro AndrewHanasiro deleted the feature/update-package branch April 15, 2026 21:04
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