Awesome Reviewers checks + prompts

sources / grafana/grafana

grafana/grafana

The checks this repository runs, scanned at d750702, and the review prompts derived from its discussions.

11 recognised tools
43 prompts
68825 stars
Observability & Telemetry
GitHub

Static analysis

3 tools

ToolKindCommandRuns inFound in
eslint Linting eslint ./ ./public/app/extensions/ --cache --no-error-on-unmatched-pattern
eslint --ext .js,.jsx,.ts,.tsx,.mjs,.cjs,.mts,.cts --cache --prune-suppressio...
eslint . --ignore-pattern *.test.ts* --ignore-pattern *.spec.ts* --cache --pl...
pre-commit, ci, local .github/workflows/frontend-lint.yml
package.json
lefthook.yml
eslint.config.js
golangci-lint Linting uses golangci/golangci-lint-action ci .github/workflows/go-lint.yml
.golangci.yml
tsc Type checking tsc --noEmit
tsc --noEmit -p scripts/rspack/tsconfig.json
tsc --noEmit -p scripts/release/tsconfig.json
pre-commit, ci, local .github/workflows/frontend-lint.yml
package.json
lefthook.yml

Testing

6 tools

ToolKindCommandRuns inFound in
ci-matrix Version and platform matrix grafana (shard: 8)
grafana-enterprise (shard: 8)
run-playwright-tests (shard: 8, shardTotal: 1)
ci .github/workflows/backend-unit-tests.yml
.github/workflows/pr-e2e-tests.yml
.github/workflows/pr-frontend-unit-tests.yml
go-test Unit and integration tests go test -v -count=1 -timeout 5m -run TestIntegrationAvailableChannels ./pkg/t...
go test -v -count=1 -timeout 5m -run TestIntegrationTypeSchemaList ./pkg/test...
go test -vet=off -short -timeout=30m ${PACKAGES[@]}
ci, local .github/workflows/alerting-update-module.yml
.github/workflows/backend-unit-tests.yml
.github/workflows/feature-toggles-ci.yml
Makefile
jest Unit and integration tests jest --ci --reporters=default --reporters=jest-junit -w ${TEST_MAX_WORKERS:-1...
jest --watch
jest --coverage
ci, local .github/workflows/pr-frontend-unit-tests.yml
package.json
Makefile
jest.config.js
node-test Unit and integration tests node --test scripts/cli/checkBundleSize.test.mjs
node --test ./scripts/cli/analytics/*.test.mts
ci, local .github/workflows/pr-bundle-size.yml
package.json
playwright End-to-end browser tests playwright test --grep-invert @cloud-plugins
playwright test --project=diagnostics --reporter=list
playwright test -c playwright.storybook.config.ts
ci, local .github/workflows/frontend-perf-tests.yaml
package.json
.github/workflows/pr-diagnostics-e2e.yml
.github/workflows/pr-e2e-tests.yml
vitest Unit and integration tests vitest run --config scripts/rspack/vitest.config.ts
vitest run --config scripts/release/vitest.config.mts
ci, local .github/workflows/pr-frontend-unit-tests.yml
package.json
scripts/release/vitest.config.mts
scripts/rspack/vitest.config.ts
ToolKindCommandRuns inFound in
gofmt Formatting gofmt -w -s {staged_files} pre-commit lefthook.yml
prettier Formatting prettier --write CHANGELOG.md
prettier --check --list-different=false --log-level=warn **/*.{ts,tsx,scss,md...
prettier --check --list-different=false --log-level=warn docs/**/*.md *.md pa...
pre-commit, ci, local .github/workflows/changelog.yml
.github/workflows/frontend-lint.yml
package.json
.github/workflows/lint-build-docs.yml
32 other commands the scanner could not classify

Commands from CI steps and task runners that look like checks but match no known tool. Some are checks, some are not.

  • DOCKER_JS_PNPM_INSTALL_FLAG = --no-frozen-lockfile
  • Team-specific review rules: Read team_rules.txt
  • bash scripts/protobuf-check.sh
  • bash scripts/verify-pkg-stig.sh dist/grafana_0.0.0_stig-test_linux_amd64.rpm
  • buf generate apps/secret --template apps/secret/buf.gen.yaml
  • command -v tparse >/dev/null 2>&1
  • docs
  • else go test ./pkg/services/featuremgmt/...
  • go get github.com/tkrajina/typescriptify-golang-structs/typescriptify@v0.1.7
  • go list -m -f {{.Dir}}
  • json2ts --ignoreMinAndMaxItems > ${PLUGIN_SCHEMA_TYPES_OUT:-./plugin-schema.d.ts}
  • license_finder --decisions-file .github/license_finder.yaml
  • madge --circular --extensions ts,tsx --no-count --no-spinner .
  • node --cpu-prof --cpu-prof-dir=.profiles --cpu-prof-name=grafana-jest-$(date -u +%Y-%m-%dT%H:%M:%S).cpuprofile node_m...
  • node ./community-triage/.github/workflows/scripts/pr-notify.mts check-teams
  • node ./pr/scripts/compare-coverage-by-codeowner.js >> $GITHUB_STEP_SUMMARY
  • node scripts/cli/checkBundleSize.mjs $RUNNER_TEMP/base-bundle-sizes.txt $RUNNER_TEMP/pr-bundle-sizes.txt $MAX_BUNDLE_...
  • node scripts/codeowners-manifest/list-tests.mts
  • node scripts/test-coverage-by-codeowner.js
  • nx run-many -t build --projects=@test-plugins/*
  • nx run-many -t dev --projects=@test-plugins/* --maxParallel=100
  • nx run-many -t test:ci --projects=tag:scope:package
  • nx run-many -t test:ci --projects=tag:scope:plugin --maxParallel=2
  • nx run-many -t typecheck --projects=tag:scope:package
  • sort -u
  • tparse -all
  • true
  • tsx ./scripts/docs/generate-transformations.ts
  • type != object)
  • xargs -I{} sh -c 'test ! -f {}/.nolint
  • xargs gofmt -s -e -l 2>&1
  • xargs gofmt -s -w 2>&1

full scan as JSON

Review prompts

most discussed first

  • grafana/grafana Code Style Go observability
    raw open
  • grafana/grafana Concurrency Go observability
    raw open
  • grafana/grafana Documentation Markdown observability
    raw open
  • grafana/grafana Null Handling Go observability
    raw open
  • grafana/grafana Naming Conventions TSX observability
    raw open
  • grafana/grafana Code Style TypeScript observability
    raw open
  • grafana/grafana Code Style Yaml observability
    raw open
  • grafana/grafana Error Handling Go observability
    raw open
  • grafana/grafana Observability Go observability
    raw open
  • grafana/grafana React TSX observability
    raw open
  • grafana/grafana Null Handling TypeScript observability
    raw open
  • grafana/grafana Configurations Go observability
    raw open
  • grafana/grafana Configurations Markdown observability
    raw open
  • grafana/grafana Performance Optimization Go observability
    raw open
  • grafana/grafana Database Go observability
    raw open
  • grafana/grafana Naming Conventions Go observability
    raw open
  • grafana/grafana Null Handling TSX observability
    raw open
  • grafana/grafana Naming Conventions TypeScript observability
    raw open
  • grafana/grafana Security Go observability
    raw open
  • grafana/grafana API Go observability
    raw open
  • grafana/grafana Testing TSX observability
    raw open
  • grafana/grafana Configurations TypeScript observability
    raw open
  • grafana/grafana Configurations Json observability
    raw open
  • grafana/grafana API TypeScript observability
    raw open
  • grafana/grafana Performance Optimization TSX observability
    raw open
  • grafana/grafana Configurations TSX observability
    raw open
  • grafana/grafana Algorithms TypeScript observability
    raw open
  • grafana/grafana Code Style Json observability
    raw open
  • grafana/grafana Documentation TSX observability
    raw open
  • grafana/grafana Code Style TSX observability
    raw open
  • grafana/grafana API TSX observability
    raw open
  • grafana/grafana Algorithms Go observability
    raw open
  • grafana/grafana Testing TypeScript observability
    raw open
  • grafana/grafana Testing Go observability
    raw open
  • grafana/grafana Performance Optimization TypeScript observability
    raw open
  • grafana/grafana Logging Go observability
    raw open
  • grafana/grafana Networking Go observability
    raw open
  • grafana/grafana Security TypeScript observability
    raw open
  • grafana/grafana Security Yaml observability
    raw open
  • grafana/grafana Caching TypeScript observability
    raw open
  • grafana/grafana Security Other observability
    raw open
  • grafana/grafana Security Markdown observability
    raw open
  • grafana/grafana Security TSX observability
    raw open