Skip to content

Commit

Permalink
fix lints
Browse files Browse the repository at this point in the history
  • Loading branch information
stanistan committed Jun 29, 2024
1 parent 093f3d9 commit 417099f
Show file tree
Hide file tree
Showing 5 changed files with 11 additions and 4 deletions.
5 changes: 5 additions & 0 deletions .golangci.yml
Original file line number Diff line number Diff line change
@@ -1,3 +1,8 @@
run:
tests: true
skip-dirs:
- internal/tools
linters-settings:
govet:
enable-all: true

2 changes: 1 addition & 1 deletion internal/github/api_source_list_sources.go
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,7 @@ func (s *ListSourcesAPISource) Sources(ctx context.Context) (api.Review, []api.S
continue
}
id := *c.PullRequestReviewID
count, _ := commentsByReviewID[id]
count := commentsByReviewID[id]
commentsByReviewID[id] = count + 1
}

Expand Down
4 changes: 3 additions & 1 deletion internal/view/review/content.go
Original file line number Diff line number Diff line change
Expand Up @@ -117,7 +117,9 @@ func (c Card) Render() el.Component {
el.Div{
el.Class("w-full", "text-sm", "p-3", "bg-slate-100", "border-b", "border-slate-300", "gap-1", "rounded-t-xl"),
badge(c.Badge),
el.Content{c.Title},
el.Content{
c.Title,
},
},
el.Content{c.Body},
},
Expand Down
2 changes: 1 addition & 1 deletion internal/view/review/page_content.go
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ func PageContent(p github.ReviewParamsMap, model api.Review) veun.AsView {
},
el.Class("underline", "hover:no-underline"),
el.Href(c.Title.HRef),
el.Attr{"target", "_blank"},
el.Attr{Key: "target", Value: "_blank"},
},
Body: el.Div{
el.Class("flex flex-col md:flex-row max-h-[95vh] bg-gray-50"),
Expand Down
2 changes: 1 addition & 1 deletion internal/view/review/slide_content.go
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@ func SlideContent(p github.ReviewParamsMap, model api.Review) veun.AsView {
Card{
Badge: idx + 1,
Title: el.A{
el.Attr{"target", "_blank"},
el.Attr{Key: "target", Value: "_blank"},
el.Href(c.Title.HRef),
el.Code{el.Text(c.Title.Text)},
},
Expand Down

0 comments on commit 417099f

Please sign in to comment.