Skip to content

fix: bump go-m1cpu to v0.1.7 to fix compiler warnings#2550

Merged
jensneuse merged 1 commit intomainfrom
jensneuse/bump-go-m1cpu
Feb 24, 2026
Merged

fix: bump go-m1cpu to v0.1.7 to fix compiler warnings#2550
jensneuse merged 1 commit intomainfrom
jensneuse/bump-go-m1cpu

Conversation

@jensneuse
Copy link
Copy Markdown
Member

@jensneuse jensneuse commented Feb 24, 2026

Summary

  • Bumps github.com/shoenig/go-m1cpu from v0.1.6 to v0.1.7 in router/ and aws-lambda-router/

Motivation

v0.1.6 produces C compiler warnings on macOS when building with cgo:

go-m1cpu@v0.1.6/cpu.go:75:17: warning: variable length array folded to constant array as an extension [-Wgnu-folding-constant]
go-m1cpu@v0.1.6/cpu.go:77:16: warning: variable length array folded to constant array as an extension [-Wgnu-folding-constant]

v0.1.7 fixes these warnings.

Test plan

  • go mod tidy succeeds in both modules
  • CI passes

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Routine dependency updates applied to maintain system stability and compatibility.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Feb 24, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 43aa77e and b85a10e.

⛔ Files ignored due to path filters (2)
  • aws-lambda-router/go.sum is excluded by !**/*.sum
  • router/go.sum is excluded by !**/*.sum
📒 Files selected for processing (2)
  • aws-lambda-router/go.mod
  • router/go.mod

Walkthrough

Two go.mod files updated to upgrade the indirect dependency github.com/shoenig/go-m1cpu from v0.1.6 to v0.1.7. No public API changes or logic modifications.

Changes

Cohort / File(s) Summary
Dependency Version Update
aws-lambda-router/go.mod, router/go.mod
Upgraded indirect dependency github.com/shoenig/go-m1cpu from v0.1.6 to v0.1.7 across both go.mod files.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: bumping the go-m1cpu dependency to v0.1.7 to resolve compiler warnings, which aligns with the changeset modifications.
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.


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

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Feb 24, 2026

Router-nonroot image scan passed

✅ No security vulnerabilities found in image:

ghcr.io/wundergraph/cosmo/router:sha-530387e33b99a98f737e9bcf1c46f1b52fbca66d-nonroot

@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 24, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 62.21%. Comparing base (43aa77e) to head (b85a10e).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2550   +/-   ##
=======================================
  Coverage   62.21%   62.21%           
=======================================
  Files         241      241           
  Lines       25499    25499           
=======================================
  Hits        15864    15864           
- Misses       8297     8298    +1     
+ Partials     1338     1337    -1     

see 2 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Copy Markdown
Contributor

@StarpTech StarpTech left a comment

Choose a reason for hiding this comment

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

LGTM

@jensneuse jensneuse merged commit 88e4878 into main Feb 24, 2026
32 checks passed
@jensneuse jensneuse deleted the jensneuse/bump-go-m1cpu branch February 24, 2026 11:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants