fix: change metrics output

[k1LoW]

• Modify the structure of tailor.Metric
• Output metrics including Coverage and Lint.

[Copilot]

Pull Request Overview

This PR modifies the structure of tailor.Metric to enhance metrics output functionality and adds new coverage and lint metrics reporting capabilities.

• Replace Description field with Key, Unit, and Error fields in the Metric struct
• Add coverage percentage and lint warnings total metrics to metrics output
• Enhance metrics command with table formatting, lint warnings display, and octocov format output support

Reviewed Changes

Copilot reviewed 6 out of 7 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
tailor/metrics.go	Updates Metric struct fields and adds coverage/lint metrics computation
tailor/metrics_test.go	Updates all test cases to use new Metric struct field names and adds coverage for new metrics
cmd/metrics.go	Enhances metrics command with table output, coverage reports, lint warnings, and octocov format support
cmd/coverage.go	Adds division by zero protection for coverage calculations
go.mod	Adds tablewriter dependency and updates existing dependencies
README.md	Updates documentation to reflect new metrics structure and command options

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[toiroakr]

What does it tests here?

If testing metric.Key is in expected key set:

expectedKeys := map[string]struct{}{
	"pipeline_resolver_step_coverage_percentage": struct{}{},
    ...
}

for _, metric := range metrics {
	_, exists := expectedKeys[metric.Key]
	if !exists {
		t.Errorf("Unexpected metric: %s", metric.Key)
	}
	// `if expectedKey != metric.Key` is not needed
}

[k1LoW]

Oh...

[k1LoW]

The purpose of the Name field had changed, making the test unnecessary.

[github-actions]

Code Metrics Report

	main (b4d7b80)	#27 (77bc9c1)	+/-
Coverage	32.5%	30.7%	-1.8%
Code to Test Ratio	1:1.7	1:1.4	-0.3
Test Execution Time	8s	3s	-5s

Details

  |                     | main (b4d7b80) | #27 (77bc9c1) |  +/-  |
  |---------------------|----------------|---------------|-------|
- | Coverage            |          32.5% |         30.7% | -1.8% |
  |   Files             |             12 |            12 |     0 |
  |   Lines             |            406 |           481 |   +75 |
+ |   Covered           |            132 |           148 |   +16 |
- | Code to Test Ratio  |          1:1.7 |         1:1.4 |  -0.3 |
  |   Code              |           1142 |          1345 |  +203 |
- |   Test              |           1970 |          1924 |   -46 |
+ | Test Execution Time |             8s |            3s |   -5s |

Code coverage of files in pull request scope (32.1% → 27.1%)

Files	Coverage	+/-	Status
cmd/coverage.go	0.0%	0.0%	modified
cmd/metrics.go	0.0%	0.0%	modified
tailor/metrics.go	93.6%	-6.4%	modified

---

Reported by octocov

[k1LoW]

@toiroakr Thank you!