Skip to content

dev/sg: rename sg check to sg lint

Warren Gifford requested to merge sg-check-to-lint into main

Created by: bobheadxi

Closes https://github.com/sourcegraph/sourcegraph/issues/31632

I don't think there are many users of check right now, so I opted to make check be a command that simply errors and tells you to use lint instead of making it work.

Test plan

~/Projects/sourcegraph/sourcegraph main*
❯ go run ./dev/sg lint -help
USAGE
  sg lint [...targets]

Run all or specified linters on the codebase and display failures, if any.

SUBCOMMANDS
  urls     Check for broken urls in the codebase.
  go       Check go code for linting errors, forbidden imports, generated files...
  docsite  Check the code powering docs.sourcegraph.com for broken links and linting errors.
  docker   Check Dockerfiles for Sourcegraph best practices
  client   Check client code for linting errors, forbidden imports, ...
  shell    Check shell code for linting errors, formatting, ...

FLAGS
  -annotations=false  Write helpful output to annotations directory


~/Projects/sourcegraph/sourcegraph main* 7s
❯ go run ./dev/sg check     
error: 'sg check' is deprecated - use 'sg lint' instead
exit status 1

Merge request reports