v0.3.0 a7c1682

ignore.go RawHistoryFolder

1package analyzer
2
3import (
4	"go/ast"
5	"go/token"
6	"strings"
7
8	"github.com/bnema/hexcheck/config"
9	"golang.org/x/tools/go/analysis"
10)
11
12// DirectiveCategory is the diagnostic category used for malformed
13// //hexcheck:ignore directives.
14const DirectiveCategory = "invalid-ignore-directive"
15
16const ignoreDirective = "//hexcheck:ignore"
17
18type ignoreKey struct {
19	file string
20	line int
21	rule string
22}
23
24// ignoreIndex records which rules are suppressed on which lines.
25type ignoreIndex map[ignoreKey]bool
26
27// collectIgnores indexes `//hexcheck:ignore rule[,rule] reason` directives.
28// A directive suppresses matching diagnostics on its own line and on the
29// following line. A reason is required, like config allow entries.
30func (r runner) collectIgnores(file *ast.File) {
31	known := config.DefaultRuleSeverities()
32	for _, group := range file.Comments {
33		for _, comment := range group.List {
34			rest, ok := strings.CutPrefix(comment.Text, ignoreDirective)
35			if !ok || (rest != "" && rest[0] != ' ' && rest[0] != '\t') {
36				continue
37			}
38			fields := strings.Fields(rest)
39			if len(fields) < 2 {
40				r.reportDirective(comment.Pos(), "hexcheck:ignore needs a rule list and a reason")
41				continue
42			}
43			pos := r.pass.Fset.Position(comment.Pos())
44			for rule := range strings.SplitSeq(fields[0], ",") {
45				if _, ok := known[rule]; !ok {
46					r.reportDirective(comment.Pos(), "hexcheck:ignore names unknown rule "+rule)
47					continue
48				}
49				r.ignores[ignoreKey{file: pos.Filename, line: pos.Line, rule: rule}] = true
50				r.ignores[ignoreKey{file: pos.Filename, line: pos.Line + 1, rule: rule}] = true
51			}
52		}
53	}
54}
55
56func (r runner) isIgnoredInline(pos token.Pos, rule string) bool {
57	if r.ignores == nil {
58		return false
59	}
60	p := r.pass.Fset.Position(pos)
61	return r.ignores[ignoreKey{file: p.Filename, line: p.Line, rule: rule}]
62}
63
64func (r runner) reportDirective(pos token.Pos, msg string) {
65	r.pass.Report(analysis.Diagnostic{Pos: pos, Category: DirectiveCategory, Message: "[error] " + DirectiveCategory + ": " + msg})
66}

Keyboard shortcuts

Single-key shortcuts are on. ctrl k and esc work whatever this says.

ctrl k :
go to a section, branch, tag, file or repository
?
this list
j k
move the row cursor down / up
enter
open the row under the cursor; on Files, a directory opens or closes in place
↑ ↓ ← → home end
in the file tree (focus on a row): previous / next row, close / open a directory or go to its parent / first entry, first / last row. They never open a file
l h
on Files: open a directory or a file / close a directory or go up to its parent, then to the parent page. Elsewhere: into the side panel (on a merge request, its files) / back to the content; there j k move between its controls
g g G
first / last row
/
filter the list; on Files, find a file anywhere in the repository (esc clears, enter opens the best match)
b
switch branch or tag
g s g f g c g b g m g i g r g a
tabs: summary, files, commits, branches & tags, merge requests, issues, releases, actions (when available)
g h
home
w
wide mode: use the whole width of the screen, or not (remembered)
y
copy a permalink pinned to the commit
d u x
diff: split / unified / collapse or expand all
e E
diff: the whole file in view / every file, with the lines the hunks hide
n
merge request: next note in the diff
esc
close a dialog, leave a field

Branches 1

Tags 5

↑↓ moveenter openesc close