feat: reimplement dropped features (reload, health, tenant guard) #401
ci.yml
on: pull_request
lint
18s
Merge Unit/CLI/BDD Coverage
16s
Annotations
10 errors
|
lint
issues found
|
|
lint:
health.go#L133
error returned from interface method should be wrapped: sig: func (github.com/GoCodeAlone/modular.HealthProvider).HealthCheck(ctx context.Context) ([]github.com/GoCodeAlone/modular.HealthReport, error) (wrapcheck)
|
|
lint:
observer.go#L98
File is not properly formatted (gofmt)
|
|
lint:
builder.go#L23
File is not properly formatted (gofmt)
|
|
lint:
health.go#L24
missing cases in switch of type modular.HealthStatus: modular.StatusUnknown (exhaustive)
|
|
lint:
reload_orchestrator.go#L116
do not define dynamic errors, use wrapped static errors instead: "errors.New(\"reload already in progress\")" (err113)
|
|
lint:
reload_orchestrator.go#L75
do not define dynamic errors, use wrapped static errors instead: "errors.New(\"reload request channel is full\")" (err113)
|
|
lint:
reload_orchestrator.go#L69
do not define dynamic errors, use wrapped static errors instead: "errors.New(\"reload circuit breaker is open; backing off\")" (err113)
|
|
lint:
reload_orchestrator.go#L93
Non-inherited new context, use function like `context.WithXXX` instead (contextcheck)
|
|
Merge Unit/CLI/BDD Coverage
Unable to download artifact(s): Artifact not found for name: merged-bdd-coverage
Please ensure that your artifact is not expired and the artifact was uploaded using a compatible version of toolkit/upload-artifact.
For more information, visit the GitHub Artifacts FAQ: https://github.com/actions/toolkit/blob/main/packages/artifact/docs/faq.md
|
Artifacts
Produced during runtime
| Name | Size | Digest | |
|---|---|---|---|
|
cli-coverage
Expired
|
13 KB |
sha256:54b0ed37208dd767cb5c8aad9e46603f513fe33e187cd94d731dcc3c7badd699
|
|
|
total-coverage
Expired
|
40 KB |
sha256:903baf4fda21fecd2103f36aa4fe586204766e63e9ea9539f91f73145884bd52
|
|
|
unit-coverage
Expired
|
27.1 KB |
sha256:cd2858ab5a7f7b10ccfef02e77fe9e1e3c02d398b337b0e8dae219a18924e9dd
|
|