Skip to content

Conversation

@alexandernorth
Copy link
Contributor

@alexandernorth alexandernorth commented Dec 11, 2025

Depends on sdcio/sdc-protos#97

Migrate multiple implementations of conversion functions to a single centralised version.

@alexandernorth alexandernorth force-pushed the feature/migrate-conversion-functions branch from f74172a to cd229aa Compare January 14, 2026 09:16
@alexandernorth alexandernorth force-pushed the feature/migrate-conversion-functions branch from 1583f2b to 7fa10fb Compare January 16, 2026 12:42
@henderiw henderiw moved this to Backlog in SDC project Jan 19, 2026
Copy link
Contributor

@henderiw henderiw left a comment

Choose a reason for hiding this comment

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

lgtm

@steiler steiler merged commit d48df65 into main Jan 20, 2026
5 of 6 checks passed
@github-project-automation github-project-automation bot moved this from Backlog to Done in SDC project Jan 20, 2026
@codecov
Copy link

codecov bot commented Jan 20, 2026

Codecov Report

❌ Patch coverage is 40.90909% with 13 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
pkg/utils/converter.go 0.00% 10 Missing ⚠️
main.go 0.00% 2 Missing ⚠️
pkg/utils/leaf_convert.go 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@alexandernorth alexandernorth deleted the feature/migrate-conversion-functions branch January 20, 2026 11:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants