Skip to content

Comments

feat(cert-tools): Add typed errors and restructure code#678

Merged
Techassi merged 5 commits intomainfrom
feat/cert-tools-improvements
Feb 18, 2026
Merged

feat(cert-tools): Add typed errors and restructure code#678
Techassi merged 5 commits intomainfrom
feat/cert-tools-improvements

Conversation

@Techassi
Copy link
Member

This PR adds a bunch of improvements to the cert-tools tool:

  • Include build-time data (like the package name)
  • Add typed error types
  • Restructure code for better clarity

@Techassi Techassi self-assigned this Feb 13, 2026
@Techassi Techassi moved this to Development: Waiting for Review in Stackable Engineering Feb 13, 2026
@adwk67 adwk67 self-requested a review February 17, 2026 15:20
@adwk67 adwk67 moved this from Development: Waiting for Review to Development: In Review in Stackable Engineering Feb 17, 2026
Copy link
Member

@adwk67 adwk67 left a comment

Choose a reason for hiding this comment

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

LGTM. Just two minor nits.

Techassi and others added 2 commits February 18, 2026 15:47
Co-authored-by: Andrew Kenworthy <andrew.kenworthy@stackable.tech>
@Techassi Techassi requested a review from adwk67 February 18, 2026 14:50
Copy link
Member

@adwk67 adwk67 left a comment

Choose a reason for hiding this comment

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

LGTM

@Techassi Techassi added this pull request to the merge queue Feb 18, 2026
@Techassi Techassi moved this from Development: In Review to Development: Done in Stackable Engineering Feb 18, 2026
Merged via the queue into main with commit b07e640 Feb 18, 2026
12 checks passed
@Techassi Techassi deleted the feat/cert-tools-improvements branch February 18, 2026 15:30
@lfrancke lfrancke moved this from Development: Done to Done in Stackable Engineering Feb 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

2 participants