Skip to content

fix(deps): require SignozSwift 0.4.2 (ENG-4529) - #35

Merged
Ryan Zhu (underthestars-zhy) merged 1 commit into
mainfrom
capy/signozswift-0.4.2
Oct 28, 2026
Merged

Ryan Zhu (underthestars-zhy) merged 1 commit into
mainfrom
capy/signozswift-0.4.2

Conversation

@underthestars-zhy

@underthestars-zhy Ryan Zhu (underthestars-zhy) commented Oct 28, 2026 •

Copy link
Copy Markdown
Member

Part of ENG-4529. Moves the SignozSwift requirement from branch: "main" to from: "0.4.2", which is SignozSwift's current production release. SwiftPM resolves a library's requirements in each consumer, so this is the requirement macrocosm-astrolabe, macrocosm-status and both cosmos-clis inherit through Astrolabe. BuildSpace's Swift promote checks a library against its Package.swift and refuses branch: "main", so this also unblocks promoting Astrolabe once #34 is in.

Package.resolved now pins SignozSwift at 0.4.2 (4b62b44). It also drops opentracing-objc and thrift-swift, which nothing requires anymore since SignozSwift switched to photon-hq/opentelemetry-swift. No other pin changes. SignozSwift's public API is unchanged since 0.2.4.

Open in Capy


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need.

Summary by CodeRabbit

  • Chores
    • Updated an internal package configuration. No user-facing changes are expected.

Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>
Copilot AI balanced review requested due to automatic review settings October 28, 2026 22:43

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

@coderabbitai

coderabbitai Bot commented Oct 28, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review.

Recent review info
Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Team
  • Run ID: 314e8c62-2f91-4f14-82b9-4c5f7f927739
Commits

Reviewing files that changed from the base of the PR and between ebc26b6 and f18a44b.

Files ignored due to path filters: 1
  • Package.resolved is excluded by !**/Package.resolved
Files selected for processing: 1
  • Package.swift

Included review availability: This review used your included allowance. 1 included review remains after this review. Your included PR review attempts over the past 7 days set your current allowance at 4 reviews per hour.

Recent review details
Additional comments: 1
Package.swift: 1

22-22: LGTM!

Walkthrough

The SignozSwift package dependency now uses version 0.4.2 as its minimum instead of tracking the main branch.

Changes: Dependency version

Layer / File(s) Summary
Set the SignozSwift version constraint
Package.swift
The dependency now requires version 0.4.2 or later instead of tracking the main branch.

Priority: Normal

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other

Suggested reviewers: zhyguol

Merge Risk: Minimal · up to f18a4

No actionable merge-blocking risk remains in this dependency-only change.

Pre-merge checks: Passed 5
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly and concisely describes the dependency change to require SignozSwift 0.4.2 and includes the relevant issue identifier.
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 1 files.

Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

A rabbit checks the version line,
0.4.2 is set in place,
No more chasing main’s branch,
The package rests within its range,
And hops along with steady pace.

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

@underthestars-zhy
Ryan Zhu (underthestars-zhy) merged commit 9dbb96e into main Oct 28, 2026
3 checks passed
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.

2 participants