Skip to content

refactor: add output package and marshaler interface#451

Merged
xnyo merged 6 commits intomainfrom
giuseppe/output-refactoring
Nov 5, 2025
Merged

refactor: add output package and marshaler interface#451
xnyo merged 6 commits intomainfrom
giuseppe/output-refactoring

Conversation

@xnyo
Copy link
Copy Markdown
Member

@xnyo xnyo commented Nov 4, 2025

Refactor of the output system for the CLI program. This allows to easily add output formats. Currently, refactored the existing CLI and JSON outputs to use the newly introduced interface. This is useful to implement output in GitHub Actions format.

Part of #450

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Nov 4, 2025

CLA assistant check
All committers have signed the CLA.

@xnyo xnyo added the enhancement New feature or request label Nov 4, 2025
@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@xnyo xnyo moved this from 📬 Triage to 🧑‍💻 In development in Grafana Catalog Team Nov 4, 2025
@xnyo xnyo marked this pull request as ready for review November 4, 2025 16:03
@xnyo xnyo requested review from a team as code owners November 4, 2025 16:03
Copy link
Copy Markdown
Contributor

@andresmgot andresmgot left a comment

Choose a reason for hiding this comment

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

Didn't test this but the code LGTM

Comment thread pkg/cmd/plugincheck2/main.go Outdated
@github-project-automation github-project-automation Bot moved this from 🧑‍💻 In development to 🔬 In review in Grafana Catalog Team Nov 5, 2025
@xnyo xnyo merged commit 1ff3cec into main Nov 5, 2025
6 checks passed
@xnyo xnyo deleted the giuseppe/output-refactoring branch November 5, 2025 09:08
@github-project-automation github-project-automation Bot moved this from 🔬 In review to 🚀 Shipped in Grafana Catalog Team Nov 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants