Skip to content
This repository has been archived by the owner on Dec 12, 2024. It is now read-only.

[chore] Fmt any and goimports #207

Merged
merged 4 commits into from
Dec 10, 2022

Conversation

andresuribe87
Copy link
Contributor

Overview

  • Added goimports to the linter.
  • Applied goimports to all files.
  • Renamed interface{} => any because it's nicer.

@codecov-commenter
Copy link

codecov-commenter commented Dec 9, 2022

Codecov Report

Merging #207 (1d67f61) into main (d51f7bf) will not change coverage.
The diff coverage is 29.41%.

@@           Coverage Diff           @@
##             main     #207   +/-   ##
=======================================
  Coverage   22.67%   22.67%           
=======================================
  Files          26       26           
  Lines        2284     2284           
=======================================
  Hits          518      518           
  Misses       1684     1684           
  Partials       82       82           
Impacted Files Coverage Δ
integration/common.go 0.00% <0.00%> (ø)
internal/did/access.go 19.35% <ø> (ø)
internal/util/util.go 0.00% <0.00%> (ø)
pkg/server/router/credential.go 3.57% <ø> (ø)
pkg/server/router/operation.go 0.00% <ø> (ø)
pkg/server/router/presentation.go 3.65% <ø> (ø)
pkg/service/operation/model.go 50.00% <ø> (ø)
pkg/service/operation/service.go 0.00% <ø> (ø)
pkg/storage/filter.go 0.00% <ø> (ø)
integration/testutil.go 29.41% <33.33%> (ø)
... and 3 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Comment on lines 13 to 14
"github.com/goccy/go-json"
cmpact "github.com/goccy/go-json"
Copy link
Member

Choose a reason for hiding this comment

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

I know you didn't add this but we don't need the alias

@andresuribe87 andresuribe87 merged commit 8f69172 into TBD54566975:main Dec 10, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants