Skip to content

Rewrite project with better internal structure#2

Merged
dzeban merged 1 commit intomasterfrom
rewrite-ready
Mar 5, 2026
Merged

Rewrite project with better internal structure#2
dzeban merged 1 commit intomasterfrom
rewrite-ready

Conversation

@dzeban
Copy link
Copy Markdown
Contributor

@dzeban dzeban commented Mar 5, 2026

To make maintenance of this project easier a rewrite is badly needed. Previous code lacks clarity and tightly-coupled. In this change a better introduced:

  • Clean stages: Load, Verify, Format.
  • Separate types that are not coupled. Certificate, Report, and Formatter handle their own parts.
  • Simpler output without unnecessary fanciness of lipgloss library.

The output format has changed but I believe it's for good: less emojis, less colors. Just a straightforward output with a touch of color for status. Tests and golden files were updated accordingly.

To make maintenance of this project easier a rewrite is badly needed.
Previous code lacks clarity and tightly-coupled. In this change a
better introduced:

* Clean stages: Load, Verify, Format.
* Separate types that are not coupled. Certificate, Report, and
  Formatter handle their own parts.
* Simpler output without unnecessary fanciness of lipgloss library.

The output format has changed but I believe it's for good: less emojis,
less colors. Just a straightforward output with a touch of color for
status. Tests and golden files were updated accordingly.
@dzeban dzeban merged commit 5a7fb21 into master Mar 5, 2026
1 check 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.

1 participant